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.

Sign up for GitHub

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

Jump to bottom

Record user.ip in API Sentry transactions #1663

Open
spalmurray-codecov opened this issue Apr 25, 2024 · 0 comments · May be fixed by codecov/gazebo#2945
Open

Record user.ip in API Sentry transactions #1663

spalmurray-codecov opened this issue Apr 25, 2024 · 0 comments · May be fixed by codecov/gazebo#2945
Labels
Dev-Ready This means the UX is reviewed and ready for prioritization scheduling.

Comments

@spalmurray-codecov
Copy link
Member

spalmurray-codecov commented Apr 25, 2024 •
edited
Loading

We were looking into whether it's possible to count unique IPs over a specific transaction type, but it seems we're not attaching the IP in a way that allows this. Sending the client's IP to Sentry as user.ip will allow us to do this kind of aggregation.
@spalmurray-codecov spalmurray-codecov added the Dev-Ready This means the UX is reviewed and ready for prioritization scheduling. label Apr 25, 2024
@rohitvinnakota-codecov rohitvinnakota-codecov linked a pull request Jun 12, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Dev-Ready This means the UX is reviewed and ready for prioritization scheduling.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Setup IP address tracking for sentry user codecov/gazebo
2 participants
@rohitvinnakota-codecov @spalmurray-codecov