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

Use with lager via raven_lager_backend #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jtmoulia
Copy link
Contributor

This may be outside of the scope of this repo, but this pull request brings raven-erlang and lager together.

I'm sure that more sentry friendly metadata can be extracted from lager, but before I drilled down I thought I'd see if this was even of interest.

Cool repo btw.

@asabil
Copy link
Owner

asabil commented Sep 2, 2013

This is great! Are you interested in continuing the integration work to see if we could extract some metadata that could be pushed to sentry?

@jtmoulia
Copy link
Contributor Author

jtmoulia commented Sep 3, 2013

Hey @asabil -- I definitely wouldn't mind! General metadata like the source PID and such should be easy, and I can follow raven_error_logger's lead.

One issue: I'd like to avoid duplicating functionality: after issuing this pull request I found https://github.com/basho/lager/blob/7af433004d846f7489fab1f705f8b06f40171bb2/src/error_logger_lager_h.erl which performs for lager what raven_error_logger does for this repo -- handling and parsing events from the error_logger.

I think raven_error_logger works great, and makes more sense than raven_lager_backend re-parsing the errors after they pass through and are munged (though very prettily :) by lager. I'm actually not sure what the best approach would be for raven_lager_backend -- perhaps skipping these error_logger -> lager messages? though this seems inconsistent.

Hope that made sense -- thoughts?

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.

2 participants