Ich speichere derzeit Fehlereigenschaften im Abschnitt extra
des Fehlertyps manuell, es wäre jedoch praktisch, wenn dies in raven-js
genauso unterstützt würde wie in raven-node
:
https://github.com/getsentry/raven-node/blob/master/lib/parsers.js#L30 -L39
Wow, ich wusste nicht einmal, dass Rabenknoten das getan hat. Hmm ... ja, ich denke, wir können die gleiche Änderung vornehmen.
Noch etwas Motivation von meiner Seite -- es gibt ein paar Stellen in unserer Codebasis, an denen wir throw new Error('msg for sentry')
mit der Absicht benachrichtigen, dass dies sowohl den Benutzer benachrichtigen wird, dass etwas schief gelaufen ist, als auch die Nachricht protokolliert und den Stack-Trace an den Wachdienst übermittelt.
Fälle, in denen wir schwere Fehler verursachen, sind in der Regel am unerwartetsten, daher könnten wir im Idealfall einige zusätzliche Daten zu den Wachpostenprotokollen hinzufügen. ZB in einem Standardzweig einer switch-Anweisung, wenn dieser Zweig unerwartet war, möchten wir einen harten Fehler machen und die Daten anhängen, die wir nicht erwartet haben, sowie jeden anderen nützlichen Kontext.
Auch wenn in den meisten Fällen der Parameter extra
einfach ist, ist es nicht ideal, sie in die Fehlermeldung einzureihen, da dann die Fehler nicht dedupliziert würden.
Siehe Konversation in Nr. 483 – diese Ausgabe habe ich vorhin total verpasst, sorry.
Hilfreichster Kommentar
Wow, ich wusste nicht einmal, dass Rabenknoten das getan hat. Hmm ... ja, ich denke, wir können die gleiche Änderung vornehmen.