Skip to content

Commit f26a732

Browse files
Ry0taKR0taK
authored andcommitted
Don't accept sandbox attribute
Because sandbox is whitelist attribute, attacker will be able to create iframe that has more permission than default. Signed-off-by: RyotaK <49341894+ry0tak@users.noreply.github.com>
1 parent 85fc41c commit f26a732

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

public/js/render.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ whiteList['style'] = []
2020
// allow kbd tag
2121
whiteList['kbd'] = []
2222
// allow ifram tag with some safe attributes
23-
whiteList['iframe'] = ['allowfullscreen', 'name', 'referrerpolicy', 'sandbox', 'src', 'width', 'height']
23+
whiteList['iframe'] = ['allowfullscreen', 'name', 'referrerpolicy', 'src', 'width', 'height']
2424
// allow summary tag
2525
whiteList['summary'] = []
2626
// allow ruby tag

0 commit comments

Comments
 (0)