Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

particulares.bancosantander.es - Firefox is an unsupported browser #94623

Open
emilio opened this issue Nov 25, 2021 — with webcompat-app · 4 comments
Open

particulares.bancosantander.es - Firefox is an unsupported browser #94623

emilio opened this issue Nov 25, 2021 — with webcompat-app · 4 comments
Assignees
Labels
browser-firefox engine-gecko The browser uses the Gecko rendering engine os-linux Issues only happening on Linux. priority-normal severity-critical The site or core functionality is unusable, or you would probably open another browser to use it. sitepatch-applied There is an UA override/intervention in place for this site type-uaoverride Require a UA override for working type-unsupported Doesn't support one or more browser version100 label for browser with version 100 in their UA string
Milestone

Comments

@emilio
Copy link

emilio commented Nov 25, 2021

URL: https://particulares.bancosantander.es/login/

Browser / Version: Firefox 96.0
Operating System: Linux
Tested Another Browser: Yes Edge

Problem type: Site is not usable
Description: Browser unsupported
Steps to Reproduce:
It's a Firefox 100 UA experiment thing. As soon as I disabled that from about:studies the site works again. Otherwise it goes to https://particulares.bancosantander.es/login/navegador-incompatible.html

Seems server-side, so probably needs contact, happy to help there.

Browser Configuration
  • gfx.webrender.all: false
  • gfx.webrender.blob-images: true
  • gfx.webrender.enabled: false
  • image.mem.shared: true
  • buildID: 20211125095031
  • channel: nightly
  • hasTouchScreen: false
  • mixed active content blocked: false
  • mixed passive content blocked: false
  • tracking content blocked: false

View console log messages

From webcompat.com with ❤️

@webcompat-bot webcompat-bot added this to the needstriage milestone Nov 25, 2021
@webcompat-bot webcompat-bot added browser-firefox engine-gecko The browser uses the Gecko rendering engine priority-normal labels Nov 25, 2021
@softvision-raul-bucata softvision-raul-bucata added the os-linux Issues only happening on Linux. label Nov 26, 2021
@softvision-raul-bucata
Copy link

@emilio Thank you for your report. I was able to reproduce the issue on other devices. The page returns an error message regarding the compatibility of the browser with UA set to Firefox 100:

Screenshot_1

Tested with:

Browser / Version: Firefox Nightly 96.0a1 (2021-11-25) (64-bit)/ Firefox UA 100
Operating System: Windows 10 PRO x64

Notes:

  1. Reproducible regardless of the status of ETP
  2. Works as expected using the latest build of Firefox Nightly (with the experiment for 100 disabled where applicable):

Screenshot_2

Moving this to NeedsDiagnosis for further investigations.

@softvision-raul-bucata softvision-raul-bucata changed the title particulares.bancosantander.es - site is not usable particulares.bancosantander.es - Firefox is an unsupported browser Nov 26, 2021
@softvision-raul-bucata softvision-raul-bucata added severity-critical The site or core functionality is unusable, or you would probably open another browser to use it. type-unsupported Doesn't support one or more browser type-uaoverride Require a UA override for working version100 label for browser with version 100 in their UA string labels Nov 26, 2021
@softvision-raul-bucata softvision-raul-bucata removed the type-uaoverride Require a UA override for working label Nov 26, 2021
@emilio
Copy link
Author

emilio commented Nov 26, 2021

I contacted them and they gave me this incidence number: 1013691034

@ksy36
Copy link
Contributor

ksy36 commented Nov 26, 2021

Thanks for contacting them @emilio.

I've noticed this is happening in Chrome Canary with version 100 as well.
There is an incorrect version detection in one of the js scripts and version is being detected as 10, instead of 100:

Screen Shot 2021-11-26 at 1 37 08 PM

This is because two chars are expected in the version instead of three

return n = t.indexOf("/", t.indexOf("firefox")),
                {
                    name: "firefox",
                    version: t.substr(n + 1, 2)
                };

in this function:

var z = function() {
            var t = navigator.userAgent.toLowerCase()
              , e = window
              , n = 0;
            if (t.indexOf("android") > -1)
                return {
                    name: "android"
                };
            switch (!0) {
            case t.indexOf("android") > -1:
                return n = t.indexOf("/", t.indexOf("android")),
                {
                    name: "android",
                    version: t.substr(n + 1, 2)
                };
            case t.indexOf("ipod") > -1 || t.indexOf("iphone") > -1 || t.indexOf("mobile") > -1:
                return n = t.indexOf("/", t.indexOf("android")),
                {
                    name: "ios",
                    version: t.substr(n + 1, 2)
                };
            case t.indexOf("edge") > -1:
                return n = t.indexOf("/", t.indexOf("edge")),
                {
                    name: "edge",
                    version: t.substr(n + 1, 2)
                };
            case t.indexOf("opr") > -1 && !!e.opr:
                return n = t.indexOf("/", t.indexOf("opr")),
                {
                    name: "opr",
                    version: t.substr(n + 1, 2)
                };
            case t.indexOf("firefox") > -1:
                return n = t.indexOf("/", t.indexOf("firefox")),
                {
                    name: "firefox",
                    version: t.substr(n + 1, 2)
                };
            case t.indexOf("chrome") > -1 && void 0 !== e.chrome:
                return t.indexOf("samsungbrowser") > -1 ? (n = t.indexOf("/", t.indexOf("samsungbrowser")),
                {
                    name: "samsungbrowser"
                }) : (n = t.indexOf("/", t.indexOf("chrome")),
                {
                    name: "chrome",
                    version: t.substr(n + 1, 2)
                });
            case t.indexOf("crios") > -1:
                return n = t.indexOf("/", t.indexOf("crios")),
                {
                    name: "chrome",
                    version: t.substr(n + 1, 2)
                };
            case t.indexOf("trident") > -1:
                return n = t.indexOf("/", t.indexOf("rv")),
                {
                    name: "ie",
                    version: t.substr(n + 1, 2)
                };
            case t.indexOf("safari") > -1:
                return n = t.indexOf("/", t.indexOf("version")),
                {
                    name: "safari",
                    version: t.substr(n + 1, 2),
                    completversion: t.substr(n + 1, 8).substring(0, t.substr(n + 1, 8).indexOf(" "))
                };
            default:
                return {
                    name: "other",
                    version: "0"
                }
            }
        };

Lets see if they can address this before Firefox reaches 100, otherwise we can ship an intervention to limit the version to be < 100.

@ksy36 ksy36 modified the milestones: needsdiagnosis, sitewait Nov 26, 2021
@ksy36 ksy36 added the action-needssitepatch This web bug needs a GoFaster site patch. label Nov 29, 2021
@ksy36
Copy link
Contributor

ksy36 commented Nov 29, 2021

@ksy36 ksy36 added the type-uaoverride Require a UA override for working label Dec 2, 2021
@denschub denschub added sitepatch-applied There is an UA override/intervention in place for this site and removed action-needssitepatch This web bug needs a GoFaster site patch. labels Dec 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser-firefox engine-gecko The browser uses the Gecko rendering engine os-linux Issues only happening on Linux. priority-normal severity-critical The site or core functionality is unusable, or you would probably open another browser to use it. sitepatch-applied There is an UA override/intervention in place for this site type-uaoverride Require a UA override for working type-unsupported Doesn't support one or more browser version100 label for browser with version 100 in their UA string
Projects
None yet
Development

No branches or pull requests

5 participants