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

Wrong ruleIndex #681

Closed
pascalberger opened this issue Jul 17, 2024 · 0 comments · Fixed by #682
Closed

Wrong ruleIndex #681

pascalberger opened this issue Jul 17, 2024 · 0 comments · Fixed by #682
Labels
Area:Reporting:Sarif Reports in SARIF format Bug
Milestone
4.7.1

Comments

@pascalberger
Copy link
Member

pascalberger commented Jul 17, 2024 •
edited
Loading

If two or more different rules containing a rule URL are reported by the same issue provider the first occurrence of the second rule will have 0 as ruleIndex instead of 1
@pascalberger pascalberger added Bug Area:Reporting:Sarif Reports in SARIF format labels Jul 17, 2024
@pascalberger pascalberger added this to the 4.7.1 milestone Jul 17, 2024
pascalberger added a commit to pascalberger/Cake.Issues that referenced this issue Jul 17, 2024
@pascalberger
(cake-contribGH-681) Set correct ruleIndex
94f49b4
@pascalberger pascalberger mentioned this issue Jul 17, 2024
Merged
pascalberger added a commit to pascalberger/Cake.Issues that referenced this issue Jul 17, 2024
@pascalberger
(cake-contribGH-681) Set correct ruleIndex
34d61a4
pascalberger added a commit that referenced this issue Jul 17, 2024
@pascalberger
(GH-681) Set correct ruleIndex (#682)
1aa313b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area:Reporting:Sarif Reports in SARIF format Bug
Projects
None yet
Milestone
4.7.1
Development

Successfully merging a pull request may close this issue.

(GH-681) Set correct ruleIndex pascalberger/Cake.Issues
1 participant
@pascalberger