sarif-web-component icon indicating copy to clipboard operation
sarif-web-component copied to clipboard

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

Open johnemau opened this issue 3 years ago • 5 comments

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: https://github.com/microsoft/sarif-web-component/blob/c21d7bd50efdf5abc661c52ee1d90455371b0a72/components/getRepoUri.ts#L30

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. image

johnemau avatar May 24 '22 21:05 johnemau

@jeffersonking, any thoughts on this?

michaelcfanning avatar May 24 '22 21:05 michaelcfanning

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.

jeffersonking avatar May 25 '22 08:05 jeffersonking

The gihub-page is fixed, thank you! image

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

johnemau avatar May 27 '22 18:05 johnemau

@jeffersonking Can you update the Azure DevOps Extension too?

carlin-q-scott avatar Oct 12 '22 19:10 carlin-q-scott

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?

jeffersonking avatar Oct 13 '22 00:10 jeffersonking