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

Tag <amp-video-iframe> not works on Safari and Chrome in iOS (version 12.3 or higher) #39676

Closed
efrensalado opened this issue Dec 7, 2023 · 2 comments · Fixed by #39894
Closed

Comments

@efrensalado
Copy link

efrensalado commented Dec 7, 2023

Description

I am trying to implement amp-video-iframe like the example in the AMP documentation, on Android (Chrome) the example works correctly, but, on devices running iOS 12.3 and above, when the example is loaded, the video tries to load the resource, but it doesn't play the src video (as it can't be loaded) and the console shows some errors related to the amp-iframe.js file
simulator_screenshot_E5C49C72-B373-4EB2-ABF1-1C2E9A4CCEAA

Related issue: issue closed

Reproduction Steps

Example in official documentation
Open in iOS (Chrome or Safari)

In debug mode, the file amp-iframe.o.1.mjs in the line 690, the code that is trying to set the attribute sandbox, is setting the value allow-presentation
screenshot-with-error

Relevant Logs

Error while parsing the 'sandbox' attribute: 'allow-presentation' is an invalid sandbox flag.
amp-iframe.js:826
[Error] Not allowed to load local resource: blob://nullhttps//amp.dev/ww.mjs.map
[Error] Not allowed to request resource
[Error] Cannot load blob://nullhttps//amp.dev/ww.mjs.map due to access control checks.
[Error] Blocked a frame with origin "https://preview.amp.dev" from accessing a frame with origin "https://amp.dev". Protocols, domains, and ports must match.
	(anonymous function) (amp-intersection-observer-polyfill-0.1.js:2:6019)
	(anonymous function) (amp-intersection-observer-polyfill-0.1.js:2:6058)
	(anonymous function) (amp-intersection-observer-polyfill-0.1.js:2:14502)
	(anonymous function) (amp-intersection-observer-polyfill-0.1.js:2:5171)
	b (amp-intersection-observer-polyfill-0.1.js:2:5527)
	g (amp-intersection-observer-polyfill-0.1.js:2:4643)
	(función anónima) (amp-intersection-observer-polyfill-0.1.js:2:15405)
	f (amp-intersection-observer-polyfill-0.1.js:2:15411)
	(anonymous function) (v0.js:2:259520)
	(anonymous function) (v0.js:2:276140)

Browser(s) Affected

Chrome, Safari

OS(s) Affected

iOS

Device(s) Affected

iPhone, iPad

AMP Version Affected

2311212202000

@erwinmombay
Copy link
Member

@powerivq would you mind triaging this?

@powerivq
Copy link
Contributor

powerivq commented Jan 17, 2024

Tested https://preview.amp.dev/documentation/examples/components/amp-video-iframe#amp=1 on ios 17 (but no repro on safari desktop) and was not able to have the video played. Bumping priority up since this appears to affect a large audience.

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

Successfully merging a pull request may close this issue.

3 participants