Bug: Latest github-pages deployment does not support Azure DevOps (ADO) line numbers (deep linking)

[johnemau]

Problem

Deep linking to the line numbers is not working in the published version of this extension (as of 5/24/2022).

I see the code to support ADO line numbers here:

sarif-web-component/components/getRepoUri.ts

Line 30 in c21d7bd

if (region?.startLine) { // lines and columns are 1-based, so it is safe to to use simple truthy checks.

However, that code is missing in the github-pages deployment on December 14th: https://github.com/microsoft/sarif-web-component/deployments/activity_log?environment=github-pages
(Checked using Edge DevTools).

Investigation

This must be a bug with github-pages or in the deployment process since github says the version deployed should support line numbers.

Ask

Can confirm the support for line numbers is deployed to github-pages and in the latest version of the extensions (Especially the ADO extension)? I would appreciate it as the feature will make following-up on reports faster!

Test

Test with the attached Results.sarif.txt file (remove .txt extension): check that the link generated by the UI is missing the line number.

[michaelcfanning]

@jeffersonking, any thoughts on this?

[jeffersonking]

Just updated github-pages. @johnemau: Is it working for you now? As usual, you may need to hard refresh the page.

The root issue here is that the github-pages does not auto deploy. And the reason for that is because we (perhaps "I"), don't want an extra github-pages commit for every normal commit. The real solution is to host this site on a separate dedicated repo. Tracking with #37.

[johnemau]

The gihub-page is fixed, thank you!

Can you also confirm that the latest version is deployed to the ADO extension? I have the same issue there.

[carlin-q-scott]

@jeffersonking Can you update the Azure DevOps Extension too?

[jeffersonking]

I think the update is in progress. @EasyRhinoMSFT has access.

@carlin-q-scott Are you using the new one (https://marketplace.visualstudio.com/items?itemName=sariftools.scans) or the old one (https://marketplace.visualstudio.com/items?itemName=sariftools.sarif-viewer-build-tab) or using the source code directly?