Skip to content

Commit b8bce5a

Browse files
committed
fix: secure PDF viewer iframe by removing unnecessary file parameter
1 parent 1f6d059 commit b8bce5a

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

lms/templates/static_pdfbook.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
<iframe
3636
title="${current_chapter.get('title')}"
3737
id="viewer-frame"
38-
src="${request.path | n}?viewer=true${viewer_params | n}"
38+
src="${request.path}?viewer=true${viewer_params}"
3939
width="856" height="1108" frameborder="0" tabindex="-1" seamless></iframe>
4040
</div>
4141
</div>
@@ -86,7 +86,7 @@
8686

8787
// Load iframe without file parameter - secure approach (first time only)
8888
$('#viewer-frame').attr({
89-
'src': '${request.path | n}?viewer=true#zoom=page-fit&disableRange=true${viewer_params | n}'
89+
'src': '${request.path | n, js_escaped_string}?viewer=true${viewer_params | n, js_escaped_string}'
9090
}).focus();
9191
});
9292
</script>

0 commit comments

Comments
 (0)