Sentry-javascript: Allow ignoreErrors to take function instead of list of regexp/strings

[kamilogorek]

ref: https://github.com/getsentry/raven-js/pull/1057

[jaydp17]

There are 2 ways we can do this.
1) Allow globalOptions.ignoreErrors to take regex/string as well as functions
2) Add something like globalOptions.shouldIgnoreError which takes a function and returns a boolean

I'd prefer the 1st option but would like to know your opinion.

[kamilogorek]

Agree that 1st option is more optimal. Please keep this comment in mind though https://github.com/getsentry/raven-js/pull/1057#issuecomment-332503372

[HeroProtagonist]

Hi,

Happy Hacktober! I will take a stab at this. PR to come in the next day or two.

Cheers!

[kamilogorek]

Hey @HeroProtagonist! This issue is already in-progress – https://github.com/getsentry/raven-js/pull/1072 :)

However, we more things to choose from:

• https://github.com/getsentry/raven-js/issues/774
• https://github.com/getsentry/raven-js/issues/1011
• https://github.com/getsentry/raven-js/issues/1036
• https://github.com/getsentry/raven-js/issues/1033
• https://github.com/getsentry/raven-js/issues/423

Feel free to ask if you need any help :)

[HeroProtagonist]

@kamilogorek , Got it! I made a PR for #774 https://github.com/getsentry/raven-js/pull/1080. Hopefully it's good :crossed_fingers: