Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Infinite redirect loop between amp page and non-amp page #5461

Closed
cuba opened this issue Jun 7, 2022 · 1 comment · Fixed by #5463
Closed

Infinite redirect loop between amp page and non-amp page #5461

cuba opened this issue Jun 7, 2022 · 1 comment · Fixed by #5463

Comments

@cuba
Copy link
Contributor

cuba commented Jun 7, 2022

Description:

Infinite redirect on the following urls

RPReplay_Final1654629321.MP4

Steps to Reproduce

  1. Enable de-amping
  2. Navigate to the url above

Actual result:
Infinite load loop between amp and non-amp page

Expected result:
Should load the page without an infinite redirect (in this case, amp should fail and load the amp page)

Reproduces how often: [Easily reproduced, Intermittent Issue]
Easily reproduced

Brave Version:
1.39

Device details:
N/A

Website problems only:

  • did you check with Brave Shields down?
    no issue with brave shields down because its caused by enabling brave shields. But this is still a de-amp issue.
  • did you check in Safari/Firefox (WkWebView-based browsers)?
    Yes. Only an iOS issue

Additional Information

This is caused by a battle between de-amping forcing a non-amp page and the server hosting the website forcing the amp page.

@cuba cuba self-assigned this Jun 7, 2022
@iccub iccub added this to the 1.39.1 milestone Jun 7, 2022
@Uni-verse
Copy link
Contributor

Uni-verse commented Jun 13, 2022

Verified PASSED using 1.39.1 (22.6.10.17)

Devices Tested on:
iPhone 12 | iOS 15.4
iPad | iOS 15.5
iPhone X iOS 14.4.2

iPhone iPad
IMG_3095 IMG_0372

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.