1951456 - Strip more parameters on audible with "Copy Clean Link"

Status: RESOLVED FIXED

(Core :: Privacy: Anti-Tracking, enhancement, P3)

Description

[Manuel Bucher [:manuel]]

Parameter to strip: same as amazon
Sites to strip this parameter from (* for all): www.audible.com
Example link to site adding such a link: https://www.audible.com/pd/The-Way-of-Kings-Audiobook/B003ZWFO7E?qid=1740685450&sr=1-5&ref_pageloadid=not_applicable&pf_rd_p=83218cca-c308-412f-bfcf-90198b687a2f&pf_rd_r=AGXCBHJ4G8FY13YFXPND&plink=26zdZpyRiJJnMAPA&pageLoadId=ETWwRaHrjOEdGbzT&creativeId=0d6f6720-f41c-457e-a42b-8c8dceb62f2c&ref=a_search_c3_lProduct_1_5
Additional information about this parameter:
Expected URL: https://www.audible.com/pd/The-Way-of-Kings-Audiobook/B003ZWFO7E

This looks like it uses the same system as amazon. So probably should use the same list. Urls can be added to it on MPL2/StripOnShare.json#109

Comment 1

[James Han]

Attachment: Bug 1951456 - Copy Clean Link: Strip query parameters on www.audible.com and international domains. r=manuel

Comment 2

[James Han]

Attachment: Bug 1951456 - Copy Clean Link: Add Audible-specific tracking parameters and international domains. r=manuel

Comment 3

[James Han]

Attachment: Bug 1951456 - Copy Clean Link: Strip query parameters on www.audible.com and international domains. r=manuel

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/5c8d4df88e1b
https://hg.mozilla.org/integration/autoland/rev/a1645211b4df
Copy Clean Link: Strip query parameters on www.audible.com and international domains. r=manuel

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/641fddc5eb8c
https://hg.mozilla.org/integration/autoland/rev/2ffec350d1c3
Revert "Bug 1951456 - Copy Clean Link: Strip query parameters on www.audible.com and international domains. r=manuel" for casuing xpcshell failure on test_validate_strip_on_share_list.js

Comment 6

[Narcis Beleuzu [:NarcisB]]

Backed out for casuing xpcshell failure on test_validate_strip_on_share_list.js

Backout link: https://hg-edge.mozilla.org/integration/autoland/rev/2ffec350d1c38d954f4f97aeedb2a0ad443d8376
Log link: https://treeherder.mozilla.org/logviewer?job_id=535061430&repo=autoland&task=NRUdlJ6tQ26NIPx5_7TLpA.0&lineNumber=2934

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/6627b507b5d3
https://hg.mozilla.org/integration/autoland/rev/a188d5d60e6b
Copy Clean Link: Strip query parameters on www.audible.com and international domains. r=manuel

Comment 8

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/a188d5d60e6b

Comment 9

[James Han]

(In reply to Narcis Beleuzu [:NarcisB] from comment #6)

Backed out for casuing xpcshell failure on test_validate_strip_on_share_list.js

Backout link: https://hg-edge.mozilla.org/integration/autoland/rev/2ffec350d1c38d954f4f97aeedb2a0ad443d8376
Log link: https://treeherder.mozilla.org/logviewer?job_id=535061430&repo=autoland&task=NRUdlJ6tQ26NIPx5_7TLpA.0&lineNumber=2934

Just clearing the needinfo flag, already landed!