search

kinbiko / bugsnag

Well-documented, maintainable, idiomatic, opinionated, and *unofficial* rewrite of the Bugsnag Go notifier
MIT License
3 stars 0 forks source link

Don’t validate API key etc. if a sanitizer is set #28

Open kinbiko opened 4 years ago

kinbiko commented 4 years ago

What problem are you trying to solve?

When setting up the notifier for tests it's annoying to have to add another 3 fields just so it passes validation.

Describe how you wish this package would help you solve the problem

You could argue that since a sanitizer might change/set these properties as well it shouldn't be necessary to set them. Documentation will have to be updated accordingly as well.

kinbiko commented 3 years ago

Another use-case, in addition to testing, is enabling a Bugsnag integration in an app optionally. For example when creating a command line tool -- the author might want the integration, but others probably don't want it (as they most likely don't use Bugsnag).