Skip to content

amp redirect cause endless load loop #23178

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

Closed
lazymonkey2 opened this issue Jun 1, 2022 · 2 comments
Closed

amp redirect cause endless load loop #23178

lazymonkey2 opened this issue Jun 1, 2022 · 2 comments
Assignees

Comments

@lazymonkey2
Copy link

Description

enable amp redirect and load this page:
https://www.ilgiorno.it/cronaca/green-pass-1.7731883
the page never stops loading. it is refreshed in an endless loop.

Steps to Reproduce

Actual result:

Expected result:

Reproduces how often:

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the nightly channel?

Other Additional Information:

  • 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?

Miscellaneous Information:

@rebron
Copy link
Collaborator

rebron commented Jun 9, 2022

Hi @424344 Can you update to our latest 1.39.120? Fix should be in that version.
Can go to brave://settings/help to get the update.

I checked the link you provided with 1.39.120 and it looks like it was working properly.

@rebron rebron self-assigned this Jun 9, 2022
@rebron rebron added the needs-more-info The report requires more detail before we can decide what to do with this issue. label Jun 9, 2022
@kjozwiak
Copy link
Member

@424344 the above issues should be resolved by either #22610 or #23320 👍 #22610 went out via 1.39.120 about a week ago. #23320 went out today on Android via 1.39.123 and will make it into the desktop release in a few days.

I also tried opening https://www.ilgiorno.it/cronaca/green-pass-1.7731883 with de-amp enabled several times and it appears that it's loading without getting stuck in the loading loop. If you're still experiencing the same issue, just re-open the issue or create a new one. Really appreciate the bug report.

@kjozwiak kjozwiak added closed/works-for-me and removed needs-more-info The report requires more detail before we can decide what to do with this issue. OS/Desktop labels Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants