Sarif report: allow file paths in file URI scheme

[theKBro]

see https://issues.jenkins.io/browse/JENKINS-73933

file paths in file uri scheme are converter to normal paths

Submitter checklist

• Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
• Ensure that the pull request title represents the desired changelog entry
• Please describe what you did
• Link to relevant issues in GitHub or Jira
• Link to relevant pull requests, esp. upstream and downstream changes
• Ensure you have provided tests - that demonstrates feature works or fixes the issue

[KalleOlaviNiemitalo]

Does this work with % encoding in file: URLs?

[theKBro]

If an exception occurs it falls back to the old value

[KalleOlaviNiemitalo]

I meant, if the SARIF file has "file:///dir/%E2%82%AC", then does this translate it to "/dir/%E2%82%AC" or to "/dir/€"?

[theKBro]

I added test data to the testcase to clarify this -> its getting translated

[theKBro]

current solution seems to work under windows only...
any proposal how to improve the code?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.59%. Comparing base (387d3db) to head (7df6f4d).
Report is 20 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #1102      +/-   ##
============================================
+ Coverage     90.58%   90.59%   +0.01%     
- Complexity     2323     2328       +5     
============================================
  Files           357      357              
  Lines          6583     6594      +11     
  Branches        676      677       +1     
============================================
+ Hits           5963     5974      +11     
  Misses          417      417              
  Partials        203      203              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[uhafner]

Thanks for starting this PR! I directly committed some changes, since I didn't find a solution without coding it directly...