Create UA override for rolb.santanderbank.com for all platforms
Categories
(Web Compatibility :: Interventions, task, P1)
Tracking
(firefox-esr6868+ fixed, firefox68 fixed, firefox69 fixed, firefox70 fixed)
People
(Reporter: ehsan.akhgari, Assigned: ksenia)
References
Details
Attachments
(1 file, 1 obsolete file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
jcristau
:
approval-mozilla-release+
jcristau
:
approval-mozilla-esr68+
|
Details | Review |
The required UA override needs to apply on all platforms. It needs to modify the default UA string to replace "Gecko" with "like Gecko". I would imagine it would include code like:
uaTransformer: (originalUA) => {
return originalUA.replace("Gecko", "like Gecko");
}
Comment 1•5 years ago
|
||
Ksenia, can you work on this please?
Comment 2•5 years ago
|
||
And the patch just needs to apply to https://rolb.santanderbank.com
, yes? (or do we possibly need/want to target it more accurately to a specific landing like https://rolb.santanderbank.com/FORPAS_ENS/ChannelDriver.ssobto
?)
Assignee | ||
Comment 3•5 years ago
|
||
There is a similar issue on another page https://rolb.santanderbank.com/ENROLL_ENS/BtoChannelDriver.ssobto?dse_operationName=EnrollmentSovereign
(from https://github.com/webcompat/web-bugs/issues/32203), and possibly more, so I think we need to apply it to the https://rolb.santanderbank.com.
Reporter | ||
Comment 4•5 years ago
|
||
(In reply to Thomas Wisniewski [:twisniewski] from comment #2)
And the patch just needs to apply to
https://rolb.santanderbank.com
, yes? (or do we possibly need/want to target it more accurately to a specific landing likehttps://rolb.santanderbank.com/FORPAS_ENS/ChannelDriver.ssobto
?)
I don't know how to answer this question to be honest, I think answering it accurately requires information that I do not possess. Based on comment 3 it appears like the issue applies to more than just the URL in comment 2. It can really affect any page that includes https://rolb.santanderbank.com/Estatico/Globales/V74/Scripts/jsDomM.bjs. That is why I suggested the entire origin, but I cannot be sure if that's enough nor sufficient...
Comment 5•5 years ago
|
||
That's fine. It sounds like Ksenia found out that we'll have to go with the full domain a comment ago.
Reporter | ||
Comment 6•5 years ago
|
||
Sounds great to me!
Assignee | ||
Comment 7•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Reporter | ||
Updated•5 years ago
|
Pushed by eakhgari@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/139206811f54 add UA for rolb.santanderbank.com r=twisniewski,rhelmer
Comment 9•5 years ago
|
||
Note here: spoke with Relman, and they think we can get this out faster via a (planned) dot release ridealong. We're gonna test if patches apply cleanly then request uplifts.
Comment 10•5 years ago
|
||
Comment on attachment 9076870 [details]
Bug 1563839 - add UA for rolb.santanderbank.com
Beta/Release Uplift Approval Request
- User impact if declined: The form looks broken in Firefox.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Low risk because it's scoped to a single domain.
- String changes made/needed: None
Comment 11•5 years ago
|
||
(we're still figuring out if we can gofaster to release before the dot release -- will update here if so)
Comment 13•5 years ago
|
||
Comment on attachment 9076870 [details]
Bug 1563839 - add UA for rolb.santanderbank.com
Adds a UA override to broken feature detection on a site. Approved for 69.0b4.
Comment 14•5 years ago
|
||
Updated•5 years ago
|
Comment 15•5 years ago
|
||
bugherder |
Comment 16•5 years ago
|
||
bugherder uplift |
Comment 17•5 years ago
|
||
Comment on attachment 9076870 [details]
Bug 1563839 - add UA for rolb.santanderbank.com
Requesting release uplift and dot-release ridealong. In the course of testing Bug 1564974 (our balrog SAO update bug), Dennis discovered Bug 1565673.
If we get release uplift, we won't break about:compat -- I've already tested in Beta. Risk should be minimal as before, PI already tested it.
Comment 18•5 years ago
|
||
Comment on attachment 9076870 [details]
Bug 1563839 - add UA for rolb.santanderbank.com
approved for 68.0.1 (desktop) and 68.1esr (desktop+fennec)
Comment 19•5 years ago
|
||
bugherder uplift |
Updated•5 years ago
|
Comment 20•5 years ago
|
||
Per discussion with jcristau, we're uplifting this to 68.0.1esr also to maintain parity with the non-ESR 68.0.1 release and hopefully avoid some confusion.
Comment 21•5 years ago
|
||
uplift |
default (68.1esr): https://hg.mozilla.org/releases/mozilla-esr68/rev/85147561a2f389ee1b3d979d291f64fb301407b1
FIREFOX_ESR_68_0_X_RELBRANCH (68.0.1esr): https://hg.mozilla.org/releases/mozilla-esr68/rev/c7c791d2632d9f80884fd0d5d884de811e1e4dea
Comment 22•5 years ago
|
||
Can "://corporate.santanderbank.co.uk/" be added to this patch or does a new bug need to be filed?
Comment 23•5 years ago
|
||
(In reply to jscher2000 from comment #22)
Can "://corporate.santanderbank.co.uk/" be added to this patch or does a new bug need to be filed?
Aw shucks. :/ We would need to file a new bug -- these patches have already landed.
Comment 24•5 years ago
|
||
I called the bank today specifically referencing this bug id -- I have an account with these guys. Hope they make a move here. I also saw that on their site that they say:
We are aware that some customers are having application issues when using Firefox v68. For the best experience, please use an alternative browser.
Hoping that they actually fix this instead of posting this notice on their site.
Updated•5 years ago
|
Updated•4 years ago
|
Comment 25•2 years ago
|
||
(In reply to (no longer active) from comment #0)
The required UA override needs to apply on all platforms. It needs to modify the default UA string to replace "Gecko" with "like Gecko". I would imagine it would include code like:
uaTransformer: (originalUA) => { return originalUA.replace("Gecko", "like Gecko"); }
(In reply to Mike Taylor [:miketaylr] from comment #1)
Ksenia, can you work on this please?
Hi Mike, i'm looking for a way to request remove this UA override for olb.santanderbank.com
What's the best way to do it?
Comment 26•2 years ago
|
||
Hi Victor, just in case: did you mean rolb.santanderbank.com
(isntead of olb
?)
Are your requesting to remove it because that specific subdomain now works without the UA override?
We also have overrides for other domains, should any of those also be reconsidered? (see https://searchfox.org/mozilla-central/source/browser/extensions/webcompat/data/ua_overrides.js#445-449)
Comment 27•2 years ago
|
||
(In reply to Thomas Wisniewski [:twisniewski] from comment #26)
Hi Victor, just in case: did you mean
rolb.santanderbank.com
(isntead ofolb
?)Are your requesting to remove it because that specific subdomain now works without the UA override?
We also have overrides for other domains, should any of those also be reconsidered? (see https://searchfox.org/mozilla-central/source/browser/extensions/webcompat/data/ua_overrides.js#445-449)
Right ROLB. Our US sites (bob.santanderbank.com & rolb.santanderbank.com) have been upgraded. The old platform was the one with compatibility issues with that header. I can reach our global team as well to confirm remove the override in those other domains you mentioned.
Comment 28•2 years ago
|
||
I can reach our global team as well to confirm remove the override in those other domains you mentioned.
That would be great, as I don't have any direct contacts at your global team. Thank you so much!
I've filed bug 1756364 to remove override for rolb.
for Firefox 99 as part of our normal release cycle.
Comment 29•2 years ago
|
||
Hi Thomas. Just got a confirmation we will be removing the UA override only for the following domains bob.santanderbank.com & rolb.santanderbank.com
Thanks
Comment 30•2 years ago
|
||
Thank you for confirming! If there is anything we can do to speed up its removal on the other domains, please let us know!
Description
•