Skip to content

Commit

Permalink
Merge pull request #14559 from Snuffleupagus/revert-9505
Browse files Browse the repository at this point in the history
Revert "Don't block origin-less blob:-URLs in hosted viewer"
  • Loading branch information
timvandermeij authored Feb 13, 2022
2 parents c37d785 + 9110210 commit 7824671
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 7824671

Please sign in to comment.