Sentry-javascript: Deshazte de ignoreErrors

Daré un tirón con 1.1

Tengo toneladas de errores que ignoro. ¿Por qué se está eliminando esto? Por ejemplo, recibía continuamente un error que finalmente remontaba a algún complemento aleatorio de Firefox que generaba un error espurio. No había nada que pudiera hacer al respecto, así que lo agregué para ignorar los errores.

@jeremyhaile ¿Podrías ignoreUrls lugar? Si no, lo guardaré. Tenía la impresión de que nadie usaba esto y solo estaba agregando hinchazón. :)

Hmm, buena pregunta. Si agrego una URL como " http://static.ak.facebook.com/ " para ignoreUrls, ¿ignora todos los errores que provienen de cualquier directorio / archivo que comience con esa URL?

ignoreUrls toma una matriz de patrones de expresiones regulares. Para que pueda hacer lo que quiera. :)

@jeremyhaile Específicamente: https://github.com/getsentry/raven-js/blob/master/src/raven.js#L374

OK - Voy a intentar eliminar ignoreErrors de nuestra configuración y ver qué errores empiezo a recibir y si ignoreUrls me permite ignorarlos o no. Te haré saber lo que encuentro.

Sí, ya me estoy encontrando con errores que no puedo (o no me importan) corregir y que no se pueden excluir usando ignoreUrls. Aquí hay uno de esos errores de IE que ya estoy recibiendo. La solución recomendada según Microsoft es "actualizar a IE8" y la URL aparece como la URL de mi página, que obviamente no quiero excluir. http://support.microsoft.com/kb/927917?wa=wsignin1.0

@jeremyhaile Bastante justo. Lo mantendré alrededor. ¡Gracias por hacérmelo saber!

No hay problema, ¡aprecia las respuestas rápidas!

Sería bueno si ignoreErrors usara expresiones regulares como ignoreUrls en lugar de cadenas de coincidencia exacta. En algunos casos, me encuentro con errores que tienen un código de error extraño adjunto que no siempre es el mismo número. O errores que contienen la misma cadena pero tienen diferentes prefijos en diferentes navegadores.

+1 a la sugerencia de @jeremyhaile , ya sea hacer que use expresiones regulares o tal vez refactorizarlo en algo como ignoreErrorPatterns ayudaría a eliminar algunos bits de ruido de extensión / inyectado por ISP que veo en nuestro sitio público.

Ruido inyectado por ISP

Eso suena bien. : +1:

Estoy a favor de las expresiones regulares. ¿Crees que tiene sentido fusionar esencialmente estas dos opciones en una que solo funcione para direcciones URL o nombres de mensajes?

¿O deberían seguir separados?

Creo que tenerlo por separado sería menos confuso; de lo contrario,
parece casi seguro que alguien abrirá un problema porque su expresión regular
coincidió más de lo que esperaban.

Me gusta la idea de cambiarle el nombre para dejar en claro que lo que estás
proporcionar es una expresión regular o comprobar explícitamente la cadena vs.
regexp para decidir si probar usando == o .test() para evitar sorpresas.

Estoy de acuerdo en mantenerlo separado: no puedo pensar en casos en los que no estaría escribiendo explícitamente una expresión regular para la URL frente al mensaje. Además, si estoy escribiendo URL regex, es ineficaz compararlo con todos los mensajes.

También me gusta su idea de verificar si es una cadena o una expresión regular antes de hacer la comparación.

Tengo un problema con ignoreErrors (¿quizás arreglado con regex?) Donde recibo cientos de ""en Sentry. Pero mi ignoreErrors está configurado en:
ignoreErrors: ['', ......]

En el servidor dice:


Hace 9 minutos javascript

¿Alguna idea de por qué esto no se filtraría? Avíseme si debo abrir una edición separada para esto. (y quizás también el problema de las expresiones regulares)

@jeremyhaile ¿Puedes abrir un tema aparte para eso?

Revertí esta advertencia de desaprobación. Es solo una mala idea. :)

¡Gracias por la entrada chicos!