Skip to content

fix: normalize Bugzilla URLs before thumb lookup#65

Open
santoslgl01-web wants to merge 1 commit into
bountysource:masterfrom
santoslgl01-web:fix/issue-10
Open

fix: normalize Bugzilla URLs before thumb lookup#65
santoslgl01-web wants to merge 1 commit into
bountysource:masterfrom
santoslgl01-web:fix/issue-10

Conversation

@santoslgl01-web
Copy link
Copy Markdown

Summary

Fixes #10

Changes

  • Normalize tracker URLs before sending them to the Bountysource thumbs API.
  • Strip URL fragments so comment anchors do not create parse retries.
  • Canonicalize Bugzilla show_bug.cgi links to ?id=<number>, including URLs where id appears after other query parameters.
  • Allow Bugzilla issue pages with id later in the query string to receive the thumbs widget.

Testing

  • node --check data/thumbs.js
  • Smoke-tested the updated Bugzilla URL-matching regex against canonical and extra-parameter Bugzilla URLs.

Bounty

This PR addresses the bounty on issue #10.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 26, 2026

CLA assistant check
All committers have signed the CLA.

@santoslgl01-web
Copy link
Copy Markdown
Author

Hi! Just checking in — please let me know if any changes are needed. Happy to update the PR. Thanks! 🙂

@santoslgl01-web
Copy link
Copy Markdown
Author

Hi! Friendly ping on this PR in case it slipped through the queue. The URL-normalization fix is ready from my side; if you'd like any adjustments or extra tests, I'm happy to update it. Thanks!

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.

ERR2 [$5]

2 participants