Closed Bug 1525839 Opened 5 years ago Closed 4 years ago

Create UA override for baidu properties

Categories

(Web Compatibility :: Interventions, enhancement, P2)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: karlcow, Assigned: denschub)

References

Details

Attachments

(2 files)

We should do this, yes.

Priority: -- → P2
Attached file GitHub Pull Request
Assignee: nobody → dschubert
Status: NEW → ASSIGNED
See Also: → 945963
Depends on: 1618128
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED

After Diana from Softvision pinged me, I did some testing again, and discovered that tests results of earlier patches were... unreliable. I don't know exactly what's going on here, but on some devices, like my x86 emulator, it works, while on others, it doesn't. I assume that Baidus server-side UA sniffing is not only looking at the browser, but is also looking at the android version, and is doing different checks based on that. My Android 7 emulator works, while the Android 10 emulator doesn't. But my debug tablet, which is an android 10 tablet, also works. Since this is server-side, I have no idea what's going on here, and I couldn't figure it out. So it's easier if we just override the entire UA string with something we can confirm working once.

Diana, this is the bug for the Baidu interventions. The goal is here not directly to fix a broken website, but rather to receive the same version than Chrome does. For some sites, we do receive a different versions without us intervening. I prepared a build that should be 100% working all the time. You can download the .apk here. Could you do me a favor and test that this works now? As I said, the goal is that we receive the same version as Chrome (the same look and feel) on the following pages:

Flags: needinfo?(diana.rus)

Hi, verified the format of the sites on Samsung Galaxy S9(Android 8), Sony Xperia Z5(Android 7) on the provided try build - Firefox Nightly 68.6a1(2020-03-04)-Build ID: 20200304090322, through comparison with Chrome browser version 80.0.3987.119

Note:

Flags: needinfo?(diana.rus)
Depends on: 1621062
Blocks: 1621062
No longer depends on: 1621062
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: