1081239 - cmbchina.com sends desktop content to Firefox Android

Status: RESOLVED FIXED

(Web Compatibility :: Site Reports, defect, P1)

Description

[Hallvord R. M. Steen [:hallvors]]

Site: https://www.cmbchina.com
cmbchina.com sends desktop content to Firefox OS

:: Steps To Reproduce

This site intends to send mobile browsers to a mobile site, but doesn't know that Firefox OS devices are smartphones. Hence the browser on Firefox OS phones get desktop content instead of mobile.

The browser detection is handled with this JavaScript inside the main page: 

<script type="text/javascript">
if(/AppleWebKit.*Mobile/i.test(navigator.userAgent) || (/MIDP|SymbianOS|NOKIA|SAMSUNG|LG|NEC|TCL|Alcatel|BIRD|DBTEL|Dopod|PHILIPS|HAIER|LENOVO|MOT-|Nokia|SonyEricsson|SIE-|Amoi|ZTE/.test(navigator.userAgent))){
	if(window.location.href.indexOf("?from=mweb")<0){		try{
			if(/Android|webOS|iPhone|iPod|BlackBerry/i.test(navigator.userAgent)){
				window.location.href="http://m.cmbchina.com/";
			}else if(/iPad/i.test(navigator.userAgent)){
			}else{
				window.location.href="http://m.cmbchina.com/"
			}
		}catch(e){}
	}
}


:: Expected Result

Mobile content

:: Actual Result

Desktop site

:: Additional Information

Software Version: 2.0
Reporter's User Agent: Mozilla/5.0 (Mobile; rv:18.1) Gecko/18.1 Firefox/18.1

Comment 1

[Karl Dubost💡 :karlcow]

In accordance to https://groups.google.com/d/msg/mozilla.compatibility/OHsR590ExWs/W3ohc41tFAAJ
Requalifying the bug, the issue is happening on Firefox Android too.

Comment 2

[Mike Taylor [:miketaylr]]

Eric, would you be able to help with outreach here?

Comment 3

[Eric Tsai (no more review request accepted)]

I will take this. BingQing, I will try to find contact window, see if this is already in your contact list.

Comment 4

[Bingqing Li]

(In reply to Eric Tsai from comment #3)
> I will take this. BingQing, I will try to find contact window, see if this
> is already in your contact list.

We're trying to find someone who can help us at CMB, but please expect some delay.

Comment 5

[Sergiu Logigan [:sergiu]]

The issue is still reproducible. Firefox does not redirect to http://m.cmbchina.com/Default.aspx as Chrome does.
Karl, can you please take a look at this one?

Comment 6

[Karl Dubost💡 :karlcow]

With a chrome Mobile UA,
accessing https://www.cmbchina.com/ 
redirects the browser to http://m.cmbchina.com/Default.aspx

With a Firefox Mobile UA,
accessing https://www.cmbchina.com/ 
leaves it as-is

The diagnostic from the initial comment still stands. It is in the home page and has not changed.
https://bugzilla.mozilla.org/show_bug.cgi?id=1081239#c0

```
<script type="text/javascript">
if (/AppleWebKit.*Mobile/i.test(navigator.userAgent) || (/MIDP|SymbianOS|NOKIA|SAMSUNG|LG|NEC|TCL|Alcatel|BIRD|DBTEL|Dopod|PHILIPS|HAIER|LENOVO|MOT-|Nokia|SonyEricsson|SIE-|Amoi|ZTE/.test(navigator.userAgent))) {
  if (window.location.href.indexOf("?from=mweb") < 0) {
    try {
      if (/Android|webOS|iPhone|iPod|BlackBerry/i.test(navigator.userAgent)) {
        window.location.href = "http://m.cmbchina.com/";
      } else if (/iPad/i.test(navigator.userAgent)) {} else {
        window.location.href = "http://m.cmbchina.com/"
      }
    } catch (e) {}
  }
}
</script>
```

They could fix the script by changing this line

    if (/AppleWebKit.*Mobile/i.test(navigator.userAgent) ||

to 

    if (/Mobile/i.test(navigator.userAgent) ||

That would take care of more mobile including Firefox Android

    "Mozilla/5.0 (Android 7.0.0; Mobile; rv:62.0) Gecko/62.0 Firefox/62.0"

Comment 7

[Firefox Bug Husbandry Bot]

See bug 1547409. Moving webcompat whiteboard tags to keywords.

Comment 8

[Raul Bucata]

The issue is still reproducible. Upon accessing the page, the desktop version of the page is received instead of the mobile version:

https://prnt.sc/mUj4FKLDs3Gm

Tested with:

Browser / Version: Firefox Nightly 100.0a1 (2015869419 -🦎100.0a1-20220317092857🦎)
Operating System: Google Pixel 3 (Android 12) -1080 x 2160 pixels, 18:9 ratio (~443 ppi density)

Comment 9

[Ksenia Berezina [:ksenia]]

The override was shipped in bug1805409 (https://github.com/mozilla-mobile/firefox-android/pull/1149/files#diff-a67cec84c97ec04fc92b45a659126eebd2240df94f1829ad767e417d749a9273), so we can probably close this.