Sentry-javascript: Befreien Sie sich von ignoreErrors

Erstellt am 12. Feb. 2013  ·  18Kommentare  ·  Quelle: getsentry/sentry-javascript

Dies ist ein Überbleibsel von vor 1.0 Tagen mit dem "Script error." Problem, aber das wurde seitdem in Rabe festcodiert. Bei allem anderen hilft es nicht wirklich.

Hilfreichster Kommentar

Ich habe diese Einstellungswarnung zurückgesetzt. Es ist einfach eine schlechte Idee. :)

Danke für den Input Jungs!

Alle 18 Kommentare

Ich reiße mit 1.1

Ich habe Tonnen von Fehlern, die ich ignoriere. Warum wird das entfernt? Zum Beispiel - ich erhielt ständig einen Fehler, den ich schließlich auf ein zufälliges Firefox-Plugin zurückführte, das einen falschen Fehler erzeugte. Es gab nichts, was ich dagegen tun konnte - also habe ich es zu ignoreErrors hinzugefügt.

@jeremyhaile Könntest du es stattdessen mit ignoreUrls umgehen ? Wenn nicht, behalte ich es in der Nähe. Ich hatte den Eindruck, dass niemand dies benutzte und es nur aufblähte. :)

Hm - gute Frage. Wenn ich eine URL wie " http://static.ak.facebook.com/ " zu ignoreUrls hinzufüge, werden dann alle Fehler ignoriert, die aus einem Verzeichnis/einer Datei stammen, die mit dieser URL beginnt?

ignoreUrls ein Array von Regex-Mustern. Sie können also tun, was Sie wollen. :)

OK - ich werde versuchen, ignoreErrors aus unserer Konfiguration zu löschen und zu sehen, welche Fehler ich erhalte und ob ignoreUrls es mir erlaubt, sie zu ignorieren oder nicht. Ich werde berichten, was ich gefunden habe.

Ja - ich stoße bereits auf Fehler, die ich nicht beheben kann (oder mich nicht darum sorge) und die nicht mit ignoreUrls ausgeschlossen werden können. Hier ist ein solcher IE-Fehler, den ich bereits erhalte. Der empfohlene Fix laut Microsoft ist "Upgrade to IE8" Und die URL wird als meine Seiten-URL angezeigt, was ich natürlich nicht ausschließen möchte. http://support.microsoft.com/kb/927917?wa=wsignin1.0

@jeremyhaile Angemessen genug. Ich behalte es in der Nähe. Danke für die Information!

Kein Problem - schätzen Sie die schnellen Antworten!

Es wäre schön, wenn ignoreErrors Regexes wie ignoreUrls anstelle von exakten Match-Strings verwenden würde. In einigen Fällen trete ich auf Fehler mit einem seltsamen Fehlercode, der nicht immer dieselbe Nummer hat. Oder Fehler, die dieselbe Zeichenfolge enthalten, aber in verschiedenen Browsern unterschiedliche Präfixe haben.

+1 an den Vorschlag von @jeremyhaile - entweder die Verwendung von Regexps oder die Umgestaltung in etwas wie ignoreErrorPatterns würde dazu beitragen, ein paar Bits von Erweiterungs- / ISP-injiziertem Rauschen zu entschärfen, die ich auf unserer öffentlichen Website sehe.

ISP-injiziertes Rauschen

Das klingt gut. :+1:

Ich bin ganz für Regexes. Halten Sie es für sinnvoll, diese beiden Optionen im Wesentlichen zu einer zusammenzuführen, die nur für URLs oder Nachrichtennamen funktioniert?

Oder sollten sie trotzdem getrennt sein?

Ich denke, es wäre weniger verwirrend, es getrennt zu haben - sonst wäre es
scheint fast sicher zu sein, dass jemand ein Problem öffnet, weil seine Regexp
stimmten mehr überein, als sie erwartet hatten.

Ich mag die Idee, es entweder umzubenennen, um klar zu machen, was du bist
Bereitstellung ist ein regulärer Ausdruck oder eine explizite Prüfung auf Zeichenfolge vs.
regexp, um zu entscheiden, ob mit == oder .test() getestet werden soll, um Überraschungen zu vermeiden.

Ich stimme zu, es getrennt zu halten - mir fallen keine Fälle ein, in denen ich nicht explizit eine Regex für die URL und die Nachricht schreiben würde. Außerdem - wenn ich URL-Regex schreibe, ist es ineffizient, sie mit jeder Nachricht zu vergleichen.

Ich mag auch Ihre Idee, zu überprüfen, ob es sich um eine Zeichenfolge oder eine Regex handelt, bevor Sie den Vergleich durchführen.

Ich habe ein Problem mit ignoreErrors (vielleicht mit Regex behoben?), bei dem ich Hunderte von "" Fehler in Sentry. Aber my ignoreErrors ist gesetzt auf:
ignorierenFehler: ['',......]

Auf dem Server heißt es:


Vor 9 Minuten Javascript

Hast du eine Idee, warum das nicht gefiltert wird? Lassen Sie mich wissen, ob ich dafür ein separates Thema eröffnen soll. (und vielleicht auch das Regex-Problem)

@jeremyhaile Kannst du dafür ein separates Thema eröffnen?

Ich habe diese Einstellungswarnung zurückgesetzt. Es ist einfach eine schlechte Idee. :)

Danke für den Input Jungs!

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen