Sentry-javascript: Se débarrasser d'ignorer les erreurs

Créé le 12 févr. 2013  ·  18Commentaires  ·  Source: getsentry/sentry-javascript

Il s'agit d'un vestige d'avant 1.0 jours avec le problème "Script error." , mais cela a depuis été codé en dur dans raven. Cela n'aide pas vraiment avec autre chose.

Commentaire le plus utile

J'ai annulé cet avertissement de dépréciation. C'est juste une mauvaise idée. :)

Merci pour la contribution les gars!

Tous les 18 commentaires

Je vais tirer avec 1.1

J'ai des tonnes d'erreurs que j'ignore. Pourquoi cela est-il supprimé ? Par exemple, j'obtenais continuellement une erreur que j'ai finalement retracée comme étant un plugin Firefox aléatoire qui a généré une fausse erreur. Je ne pouvais rien y faire - je l'ai donc ajouté à ignoreErrors.

@jeremyhaile Seriez-vous capable de le contourner avec ignoreUrls place ? Sinon, je le garderai. J'avais l'impression que personne ne l'utilisait et que cela ne faisait qu'ajouter du ballonnement. :)

Hum - bonne question. Si j'ajoute une URL telle que " http://static.ak.facebook.com/ " à ignoreUrls, ignore-t-il toutes les erreurs provenant de n'importe quel répertoire/fichier commençant par cette URL ?

ignoreUrls prend un tableau de modèles de regex. Donc tu peux faire ce que tu veux. :)

OK - Je vais essayer de supprimer ignoreErrors de notre configuration et voir quelles erreurs je commence à recevoir et si ignoreUrls me permet de les ignorer ou non. Je vous dirai ce que je trouve.

Ouais - Je rencontre déjà des erreurs que je ne peux pas (ou ne me soucie pas) corriger et qui ne peuvent pas être exclues en utilisant ignoreUrls. Voici une telle erreur IE que je reçois déjà. Le correctif recommandé selon Microsoft est "Mise à niveau vers IE8" Et l'URL apparaît comme l'URL de ma page, ce que je ne veux évidemment pas exclure. http://support.microsoft.com/kb/927917?wa=wsignin1.0

@jeremyhaile Assez bien. Je vais le garder. Merci de me le faire savoir!

Pas de problème - appréciez les réponses rapides !

Ce serait bien si ignoreErrors utilisait des expressions régulières comme ignoreUrls au lieu de chaînes de correspondance exactes. Dans certains cas, je rencontre des erreurs auxquelles est joint un code d'erreur étrange qui n'est pas toujours le même numéro. Ou des erreurs qui contiennent la même chaîne mais ont des préfixes différents dans différents navigateurs.

+1 à la suggestion de ignoreErrorPatterns aiderait à débruiter quelques bits de bruit d'extension / FAI injecté que je vois sur notre site public.

Bruit injecté par le FAI

Ça sonne bien. :+1:

Je suis tout pour les regex. Pensez-vous qu'il soit logique de fusionner essentiellement ces deux options en une seule qui ne fonctionne que pour les URL ou les noms de message ?

Ou devraient-ils toujours être séparés?

Je pense qu'il serait moins déroutant de le séparer - sinon il
semble presque certain que quelqu'un ouvrira un problème parce que son regexp
correspondait plus qu'ils ne s'y attendaient.

J'aime l'idée de le renommer pour qu'il soit clair que ce que vous êtes
fournir est une expression régulière ou vérifier explicitement la chaîne vs.
regexp pour décider s'il faut tester en utilisant == ou .test() pour éviter les surprises.

Je suis d'accord pour le garder séparé - je ne peux pas penser à des cas où je n'écrirais pas explicitement une expression régulière pour l'URL par rapport au message. De plus, si j'écris une regex d'URL, il est inefficace de la comparer à chaque message.

J'aime aussi votre idée de vérifier s'il s'agit d'une chaîne ou d'une expression rationnelle avant de faire la comparaison.

J'ai un problème avec ignoreErrors (peut-être corrigé avec regex ?) où je reçois des centaines de "" erreurs dans Sentry. Mais mon ignoreErrors est défini sur :
ignorer les erreurs : ['',......]

Sur le serveur c'est écrit :


il y a 9 minutes javascript

Une idée de pourquoi cela ne serait pas filtré ? Faites-moi savoir si je dois ouvrir un sujet séparé pour cela. (et peut-être aussi le problème de regex)

@jeremyhaile Pouvez-vous ouvrir un problème distinct pour cela?

J'ai annulé cet avertissement de dépréciation. C'est juste une mauvaise idée. :)

Merci pour la contribution les gars!

Cette page vous a été utile?
0 / 5 - 0 notes