Skip to content

bug-2060967: link crash stack source files to searchfox instead of github - #7248

Merged
smarnach merged 1 commit into
mozilla-services:mainfrom
mstange:searchfox-links
Aug 26, 2026
Merged

bug-2060967: link crash stack source files to searchfox instead of github#7248
smarnach merged 1 commit into
mozilla-services:mainfrom
mstange:searchfox-links

Conversation

@mstange

@mstange mstange commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

https://bugzilla.mozilla.org/show_bug.cgi?id=2060967

Ever since bug 1987761 landed, Firefox symbol files now contain git paths like git:github.com/mozilla-firefox/firefox:dom/base/StructuredCloneBlob.cpp:ed6cfb3b73cb602ed2b8a75b1721c95b8e820322. As requested in bug 2060967, let's make those link to Searchfox. Git paths to other repos, such as Rust stdlib code, will still link to github.

…thub

Firefox symbol files now use git paths like git paths in the mozilla-firefox/firefox
repository, so stack frames linked to github.com. Searchfox is preferable for
reading crash stacks, so map that repository to searchfox instead.

All of searchfox's firefox-* trees share a single clone which has refs for
main, beta, release and the esr branches, and the /rev/ endpoint resolves any
commit in that clone. Since mozilla-firefox/firefox holds every release
branch, the firefox-main tree covers all channels.

To express a mapping for one repository rather than a whole server,
enhance_frame() now looks up the full root before falling back to the server,
which leaves other github.com repositories pointing at github. While here,
apply the 'show the basename' fallback whenever no mapping is found instead of
only when the vcs type is unknown; previously a git: path with an unrecognized
server rendered the raw 'git:server/repo:file:rev' string in the stack table.

@smarnach smarnach left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this looks good to me. I can't verify that this will result in correct Searchfox links in all cases, but I'm happy to try it.

@smarnach
smarnach added this pull request to the merge queue Aug 26, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Aug 26, 2026
@smarnach
smarnach added this pull request to the merge queue Aug 26, 2026
Merged via the queue into mozilla-services:main with commit d1541f2 Aug 26, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants