[Follow up to #22610] Auto-redirect AMP pages - Loops - page won't load on Android. - Githubissues

brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.

https://brave.com

Mozilla Public License 2.0

17.52k stars 2.27k forks source link

[Follow up to #22610] Auto-redirect AMP pages - Loops - page won't load on Android. #23312

Closed Uni-verse closed 2 years ago

Uni-verse commented 2 years ago

Description

Loading this ULR causes a AMP redirect loop, page won't load, DevTools network shows looping pageload /amp

Follow up to https://github.com/brave/brave-browser/issues/22610 as this is behaving differently on Android vs Desktop

Steps to reproduce

Open URL https://t.co/0Ydr4haUQd which should land at
https://www.airlive.net/new-footage-shows-an-ukrainian-mig-29-taking-off-from-mykolaiv-seconds-before-the-runway-is-shelled-by-the-russians/
Watch the Omnibox blink
Devtools watch the Network tab and see that it loads page then /amp and loops on that

Actual result

Page load gets stuck in loop

https://user-images.githubusercontent.com/17885425/172485641-9bbcc052-edfa-4eb3-98e4-7a6b014264aa.mp4

Expected result

Page should load

Issue reproduces how often

Easily

Version/Channel Information:

Can you reproduce this issue with the current Play Store version?
Can you reproduce this issue with the current Play Store Beta version?
Can you reproduce this issue with the current Play Store Nightly version?

Device details

Install type (ARM, x86): ARM
Device type (Phone, Tablet, Phablet): Samsung Galaxy Devices
Android version: 12

Brave version

version 1.39.118, Chromium 102.0.5005.78

Website problems only

Does the issue resolve itself when disabling Brave Shields?
Does the issue resolve itself when disabling Brave Rewards?
Is the issue reproducible on the latest version of Chrome?

Additional information

ShivanKaul commented 2 years ago

Dupe of https://github.com/brave/brave-browser/issues/23320. The site is doing a js redirect now.