Skip to content
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

Auto redirecting from AMP to non-AMP allowed? #24486

Open
AndrewKGuan opened this issue Sep 12, 2019 · 4 comments
Open

Auto redirecting from AMP to non-AMP allowed? #24486

AndrewKGuan opened this issue Sep 12, 2019 · 4 comments
Assignees
Labels

Comments

@AndrewKGuan
Copy link

AndrewKGuan commented Sep 12, 2019

Using an amp-iframe with the sandbox attribute, one can give his iframe the "maximum sandboxed" privileges, which also include the privilege to redirect. So that means one can build a contentless temporary splash screen using AMP and then redirect the user to their site from the cache-served AMP viewer.
I personally have no opinion but just want to know, should that be ok?

@cramforce
Copy link
Member

I think amp-iframe should rewrite allow-top-navigation as allow-top-navigation-with-user-interaction where supported (most browsers now).

@cramforce
Copy link
Member

To @kristoferbaxter for prioritization

@nainar nainar added P2: Soon Component: amp-iframe Customer: MSC Mobile Solution Consultants Type: DevX issues impacting developer experience WG: components labels Sep 12, 2019
@stale
Copy link

stale bot commented Mar 5, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the Stale Inactive for one year or more label Mar 5, 2021
@kristoferbaxter kristoferbaxter removed the Stale Inactive for one year or more label Mar 5, 2021
@kristoferbaxter
Copy link
Contributor

I missed this issue, apologies. Taking a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants