Skip to content

Commit ad32d3d

Browse files
authored
Merge pull request #444 from ethereum/yann300-patch-9
revert allow_same_origin
2 parents 0f0e869 + b89cb87 commit ad32d3d

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

packages/engine/web/src/lib/iframe.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,9 @@ export class IframePlugin extends PluginConnector {
1414
private origin: string
1515
private source: Window
1616
private url: string
17-
private allowSame0rigin: boolean
1817

19-
constructor(public profile: IframeProfile, private allowSame0rigin: boolean = true) {
18+
constructor(public profile: IframeProfile) {
2019
super(profile)
21-
this.allowSame0rigin = allowSame0rigin
2220
}
2321

2422
/** Implement "activate" of the ViewPlugin */
@@ -75,7 +73,7 @@ disconnect() {
7573
if (this.iframe.contentWindow) {
7674
throw new Error(`${this.name} plugin is already rendered`)
7775
}
78-
this.iframe.setAttribute('sandbox', `allow-popups allow-scripts ${this.allowSame0rigin ? 'allow-same-origin' : ''} allow-forms allow-top-navigation`)
76+
this.iframe.setAttribute('sandbox', 'allow-popups allow-scripts allow-same-origin allow-forms allow-top-navigation')
7977
this.iframe.setAttribute('seamless', 'true')
8078
this.iframe.setAttribute('id', `plugin-${this.name}`)
8179
this.iframe.src = this.url

0 commit comments

Comments
 (0)