Iframe allow attribute not working

Fonctionnalité
1

I have added a codesandbox iframe with different attributes but Discourse is just getting the src. Should I change something in Discourse Settings? I have already allowed this iframe and it is showing, but not correctly.

This is the iframe I am trying to add:

<iframe 
  src="https://codesandbox.io/embed/codesandbox-frontity-rnclp?fontsize=14" 
  title="Frontity - Post with recompose" 
  allow="geolocation; microphone; camera; midi; vr; accelerometer; gyroscope; payment; ambient-light-sensor; encrypted-media; usb" 
  style="width:100%; height:500px; border:0; border-radius: 4px; overflow:hidden;" 
  sandbox="allow-modals allow-forms allow-popups allow-scripts allow-same-origin"
></iframe>

But it is just rendering this:

Need_to_integrate_react-stripe-elements_-%F0%9F%A4%97_Dev_Talk___Questions-_Frontity_Community_Forum

So the iframe is not showing correctly:

09

3 « J'aime »
2

It works for me

image
image.png1268×70 6.02 KB

Just to be sure, did you add the iframe url (https://codesandbox.io/embed/codesandbox-frontity-rnclp) in whitelist? If not, search the site setting allowed iframes.

1 « J'aime »
3

I have this domain: https://codesandbox.io/. Shouldn’t be enough? I have found this code at Discourse where it whitelists just some iframe attributes:

If I include the attributes height and weight the iframe works fine, but I would like to allow the other attributes too.

Is there a way of allowing all codesandbox iframes? I thought including just the domain it would work.

Thank you!

EDIT:

I have tried adding this url in whitelist but it is not working neither.

3 « J'aime »
4

I am also running into this problem. I would like to add a class on the IFRAME that I am embedding in my policy privacy post, which embeds the privacy tracking settings from our self-hosted Matomo installation. This would allow me to add a better border and some color to differentiate it from the rest of the privacy policy.

Despite having a class="foo" in my IFRAME element, it is being stripped out, apparently by the white-lister code above. Any chance this could be expanded to have a few more attributes allowed?

5

For security we are very strict of what HTML attributes can be rendered in our page when they come from user input.

What you can do to get that is:

<div data-my-special-attr="42">
  <iframe src="http://example.com">
</div>

and then target that with your CSS / JS selector:

div[data-my-special-attr="42"] > iframe {
  border-color: pink;
}
3 « J'aime »
6

Thanks! That worked on both the topic page and the rendered privacy policy special page as well.

Of note if anyone else is trying this, you can’t add a class attribute, you literally need to add an attribute along the lines of data-foo-attr.

1 « J'aime »
7

Rafael thanks for the statement, it clarifies my observed behaviour.
I would like to know whether you have any plans of releasing that lock for audio/video attributes of an iframe. Modern browsers manage accessibility quite good for those allowances, and there are increasingly interesting service offerings which would be great to integrate by users but just lack this type of accessibility.
Thanks.

1 « J'aime »
8

We are always open to feedback!

It gets easier if we talk about specifics, like what are the exact attributes you want allowlisted and for what services they are necessary.

2 « J'aime »
9

ok. I especially think about Jam Systems / Jam · GitLab which would need an allow=“microphone *;” parameter to work properly.

10

I am running into same issue attempting to embed an H5P audio recorder; the allow=“microphone *;” option is stripped from the iframe.

What would take to perhaps have an iframe setting to allow allows?

11

Pouvons-nous autoriser les administrateurs à spécifier quels attributs iframe ils souhaitent autoriser ?

1 « J'aime »
13

Ce serait utile, mais nous serions également satisfaits si l’attribut allow était mis sur liste blanche pour tous. Nous rencontrons actuellement des problèmes de lecture audio avec les lecteurs de podcasts intégrés d’Apple et de Spotify. Comme d’autres l’ont mentionné, le problème est que l’attribut allow est supprimé, ce qui contient une directive importante encrypted-media.

1 « J'aime »
15

Étant donné que nous sommes déjà stricts quant aux domaines autorisés dans les iframes, avoir un autre paramètre où nous définissons la chaîne allow pour chaque iframe et analysons le format étrange du contenu allow me semble un peu excessif.

J’ai créé une PR qui permet simplement d’utiliser n’importe quoi dans l’attribut allow pour les iframes déjà autorisées :

Qu’en penses-tu @sam ?

3 « J'aime »
16

Hmm, je suppose que ça me convient, je vois ta logique ici, @david des objections ?

1 « J'aime »
17

Ce changement a été fusionné @santosguillamot @selfscrum @cogdog @gilby

3 « J'aime »