Skip to content

chore(detectors): Normalize evidence data for query injection issues #93760

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

Merged
merged 1 commit into from
Jun 18, 2025

Conversation

roggenkemper
Copy link
Member

since sql injection and query injection detectors create the same issue type, the issue details page should work for both. this pr normalizes the evidence data so the issue details page will more easily show the data for this type of issue. including the value as well, instead of just the key, to provide more evidence to the user.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 17, 2025
@roggenkemper roggenkemper marked this pull request as ready for review June 17, 2025 22:28
@roggenkemper roggenkemper requested review from a team as code owners June 17, 2025 22:28
@roggenkemper roggenkemper merged commit 790ac8d into master Jun 18, 2025
65 checks passed
@roggenkemper roggenkemper deleted the roggenkemper/normalizequeryoutput branch June 18, 2025 14:41
andrewshie-sentry pushed a commit that referenced this pull request Jun 19, 2025
…93760)

since sql injection and query injection detectors create the same issue
type, the issue details page should work for both. this pr normalizes
the evidence data so the issue details page will more easily show the
data for this type of issue. including the value as well, instead of
just the key, to provide more evidence to the user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants