Skip to content

Commit

Permalink
Explain how to comply with the conditions about using iFrames in a mo…
Browse files Browse the repository at this point in the history
…dule
  • Loading branch information
Quetzacoalt91 committed Sep 17, 2019
1 parent 2ce98d0 commit 2b5236f
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/content/1.7/modules/sell/techvalidation-keysteps.md
Expand Up @@ -34,7 +34,10 @@ After meeting each requirement provided by the Validator, here's a list of extra
* The presence of DROP/ALTER of PrestaShop tables.
* It's highly forbidden to apply any changes on PrestaShop Core tables. It's very very dangerous and we don't allow any risk.

* Except for [Payment Modules](https://github.com/PrestaShop/paymentexample/blob/master/paymentexample.php#L150), the use of iframes is STRICTLY FORBIDDEN.
* The use of iframes is highly discouraged for security reasons, although their possible implementation in different part of the core such as with [Payment Modules](https://github.com/PrestaShop/paymentexample/blob/master/paymentexample.php#L150).
* Using an iframe authorizes to load content from a site that is not controlled by us. This is the same problem as authorizing to load javascript file from an external source. If you're being hacked, the attacker could potentially exploit other failures to take control of all the shops that would have installed the module.

* Therefore we need to check what your processes are, to ensure the security of the content that will be injected by this iframe into all the shops that will install the module. When submitting your module, the validation team will review the reasons why an iFrame is needed for this business and what are the measures taken by the provider to prevent attacks.

* Every hook. If any of them is empty, we decline the zip.
* There are other things we decline like loading a JS file in the whole back office when it is unnecessary. Make sure to only load what you need, when you need it.
Expand Down

0 comments on commit 2b5236f

Please sign in to comment.