Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Errbit self-hosted exception management #3624

Merged
merged 4 commits into from
Aug 27, 2020

Conversation

javierm
Copy link
Member

@javierm javierm commented Jun 21, 2019

References

Background

We're currently using the rollbar gem, so anyone can handle their exceptions with Rollbar.

However, managing exceptions with Rollbar means sending data about your application to a third party, which is something some users might be concerned about.

Objectives

  • Add support for self-hosted exception mangement with Errbit

Notes

  • To install and configure Errbit, check their installation and notes on deployment documentation
  • The airbrake gem also works with Airbrake's hosting service, the same way the rollbar gem works with Rollbar's services

With Errbit, you can set up your own server and host the information
regarding your exceptions there. You can also hire Airbrake's hosting
services or easily setup Errbit on Heroku.

We're still including the rollbar gem so we don't harm CONSUL users who
are using rollbar.

Note Errbit requires an old version of Airbrake which forced users to
configure the gem. So we're adding the current environtment to
`ignore_environments` when the project id isn't defined; this way the
application won't crash in this case.
Since some people hosting errbit might be using it as an internal tool
with a self signed certificate, we need to patch Airbrake so it accepts
these certificates.
microweb10 and others added 2 commits August 27, 2020 16:22
Many users who didn't configure rollbar were getting exceptions when
trying to connect to the rollbar service.
@javierm
Copy link
Member Author

javierm commented Aug 27, 2020

Travis failure is not related to this pull request and will be addressed shortly.

@javierm javierm merged commit 7ee2f15 into master Aug 27, 2020
Consul Democracy automation moved this from Testing to Release 1.2.0 Aug 27, 2020
@javierm javierm deleted the add_errbit_support branch August 27, 2020 14:53
@javierm javierm removed the 1.2 label Aug 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants