Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Need to be able to set `sandbox` property to empty string

See original GitHub issue

I have an application using this component which renders a PDF inside an IFRAME. This component seems to set the sandbox property (at least in Chrome) to be wide open:

sandbox="allow-forms allow-pointer-lock allow-popups allow-same-origin allow-scripts allow-top-navigation"

I had to remove this component and replace it with a basic HTML <iframe> tag to get the PDF to render so I could set the sandbox attribute to be blank.

Help?

Issue Analytics

State:
Created 5 years ago
Comments:6 (2 by maintainers)

Top GitHub Comments

1reaction

robbestadcommented, Sep 24, 2018

Good idea, will implement!

man. 24. sep. 2018 kl. 19:18 skrev gtwilliams03 notifications@github.com:

[Edit] This does allow setting the sandbox property to “something” but it does not allow the value to be empty. I question whether a blank sandbox value should be the default?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/svenanders/react-iframe/issues/35#issuecomment-424054627, or mute the thread https://github.com/notifications/unsubscribe-auth/ABfQp4rdoYGl7ijzzh1IQ4WvYpfXamVMks5ueRP_gaJpZM4W3GBa .

– Mvh, Sven Anders Robbestad https://www.robbestad.com http://www.robbestad.com/

0reactions

gtwilliams03commented, Sep 27, 2018

I updated and all is well with me! Many, many thanks!