-
-
Notifications
You must be signed in to change notification settings - Fork 495
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
rails: allow showing link to sentry page on error page #1846
Comments
I like the idea but I'm not sure if we need to use Fiber locals for that. Won't storing it in |
storing in env will work if the core middleware is refactored to pass the env to capture_exception |
I just did it in #1847 😄 |
awesome ... so do you want to add this feature too then ? |
I'm not convinced that we should maintain a new middleware + a template (even just minimum) for this. I think as an SDK, it should just inject the event id into the |
I'm fine with that too, good step forward and easy to do ... |
Ok I’ll open a PR for that 👍 |
PR opened: #1849 |
Describe the idea
Why do you think it's beneficial to most of the users
Possible implementation
https://github.com/grosser/sentry-user_informer
The text was updated successfully, but these errors were encountered: