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

Integrate with sentry #12

Open
cdevienne opened this issue Oct 13, 2017 · 1 comment
Open

Integrate with sentry #12

cdevienne opened this issue Oct 13, 2017 · 1 comment

Comments

@cdevienne
Copy link
Member

When implementing panics handling, it would be nice to have the option to report them to a sentry instance.

@mdevan
Copy link
Member

mdevan commented Oct 16, 2017

The user services built using nRPC could use this too. Also a few other such "core" services:

  • logging - nats.Publish() your logs to a "log" service
  • application metrics - publish your metrics, or discover pull endpoints for them
  • tracing - be able to tag the reported logs or panics or metrics with a request tracking ID

Basically, my point is that there are a set of few "core" services that are must-have/useful for anyone using nRPC -- if we don't provide them, the users will have to invent it themselves. We should structure these as a "nrpc-core" package which includes .proto, generated clients and simple/standard server implementations.

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

No branches or pull requests

2 participants