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

Screen Capture API request broken inside an iframe #146

Open
tom2bec opened this issue Sep 27, 2021 · 3 comments
Open

Screen Capture API request broken inside an iframe #146

tom2bec opened this issue Sep 27, 2021 · 3 comments

Comments

@tom2bec
Copy link

tom2bec commented Sep 27, 2021

Look like Screen Capture API permission check ☝️ is now mandatory since : Chrome 94, FireFox 92 and Safari when your app is embedded inside an iframe ...

General documentation about this subject, doesn't seem to be up to date as it's not really considered anymore like an experimental feature by last browsers updates 😄

Could you consider to add parameter allow=display-capture on the iframe declaration containing third part apps inside Zendesk support ?

@sostopher
Copy link

sostopher commented Sep 27, 2021

Hi Tom,

Thanks for opening this issue. On this, we are not planning to support this parameter until it's released and in production (as per our Security team). Currently, MDN has display-capture as experimental and not recommended for production use: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Feature-Policy/display-capture

Once this changes and browsers support it (and it's no longer an experiment) we see no reason not to implement this permission.

Chris

@tom2bec
Copy link
Author

tom2bec commented Sep 28, 2021

Hey Chris,

Thanks for the quick feedback. Definitely understand that this permission is still tagged as experimental by MDN, but microphone and camera permissions are also in the same level.

Yet these two features policy are already integrated on Zendesk iframe permissions. Maybe I missed something but not sure to follow the logic ?

Note: According to MDN doc an item is no longer experimental when It is implemented in two or more major browsers : https://developer.mozilla.org/en-US/docs/MDN/Guidelines/Conventions_definitions#experimental

@tom2bec
Copy link
Author

tom2bec commented Oct 4, 2021

We've reported the point to MDN for update : mdn/browser-compat-data#12708

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

No branches or pull requests

2 participants