Skip to content

Commit

Permalink
Revert "Don't block origin-less blob:-URLs in hosted viewer"
Browse files Browse the repository at this point in the history
This reverts commit a6aca3c, since no version of Internet Explorer is supported any more.
  • Loading branch information
Snuffleupagus committed Feb 12, 2022
1 parent e9fd67a commit 9110210
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions web/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2127,14 +2127,11 @@ if (typeof PDFJSDev === "undefined" || PDFJSDev.test("GENERIC")) {
// Hosted or local viewer, allow for any file locations
return;
}
const { origin, protocol } = new URL(file, window.location.href);
const fileOrigin = new URL(file, window.location.href).origin;
// Removing of the following line will not guarantee that the viewer will
// start accepting URLs from foreign origin -- CORS headers on the remote
// server must be properly configured.
// IE10 / IE11 does not include an origin in `blob:`-URLs. So don't block
// any blob:-URL. The browser's same-origin policy will block requests to
// blob:-URLs from other origins, so this is safe.
if (origin !== viewerOrigin && protocol !== "blob:") {
if (fileOrigin !== viewerOrigin) {
throw new Error("file origin does not match viewer's");
}
} catch (ex) {
Expand Down

0 comments on commit 9110210

Please sign in to comment.