Closed Bug 1310951 Opened 3 years ago Closed 2 years ago

Create UA override for m.canadiantire.ca

Categories

(Web Compatibility :: Tooling & Investigations, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: adamopenweb, Assigned: adamopenweb)

References

()

Details

Attachments

(1 file, 2 obsolete files)

We should create a UA override for m.canadiantire.ca in fennec as mentioned here: https://webcompat.com/issues/2315#issuecomment-254447551

Spoofing the user agent makes the site work properly for us. Tested using this UA:

Mozilla/5.0 (Linux; Android 6.0.1; Nexus 6 Build/MMB29X) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.89 Mobile Safari/537.36
Issue still exists can't use in Fennec. Canadiantire.ca is ranked 83 in Canada on Alexa.
Using the same user agent string as the Direct TV entry.
Assignee: nobody → astevenson
Attachment #8868736 - Flags: review?(miket)
Attachment #8868750 - Flags: review?(miket)
Comment on attachment 8868750 [details] [diff] [review]
Adding m.canadiantire.ca override

Review of attachment 8868750 [details] [diff] [review]:
-----------------------------------------------------------------

r=me

(Note: We need to add r=miketaylr at the end of the commit message before checking it in.)
Attachment #8868750 - Flags: review?(miket) → review+
Adam, can you tweak the commit message then add the "checkin-needed" keyword?
Flags: needinfo?(astevenson)
> add the "checkin-needed" keyword

(To be clear, that keyword gets added to this bug.)
Attachment #8868750 - Attachment is obsolete: true
Flags: needinfo?(astevenson)
Attachment #8873548 - Flags: review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ecad8cb9f20c
Add UA override for m.canadiantire.ca. r=miketaylr
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/ecad8cb9f20c
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Mike, I tested this but we're still getting blocked in fennec. Can you take a look at it please?
Flags: needinfo?(miket)
Thanks Adam. I filed 1376569 to see what's up with the CDN not syncing.
Let's re-open until the fix is deployed to users.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
This has been fixed for a while. Forgot to update.
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
See Also: → 1457514
You need to log in before you can comment on or make changes to this bug.