Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable cross-origin check #16

Closed
wants to merge 1 commit into from
Closed

Conversation

MattFenelon
Copy link

The viewer for PDF.js does its own cross-origin checks, refusing to load any remote PDFs, which stops the viewer loading PDFs from CDNs or remote storage locations, such as S3.

The viewer for PDF.js does its own cross-origin checks, refusing to load any remote PDFs, which stops the viewer loading PDFs from CDNs or remote storage locations, such as S3.
@siegy22
Copy link
Collaborator

siegy22 commented Jul 28, 2016

I don't think it's the way to comment out the whole validation code.

I think the solution would be:

var HOSTED_VIEWER_ORIGINS = ['*'];

(at line 7145)

Or delete the all of the validations, but I think there's a reason why this is in here.

@senny What do you think?

@senny
Copy link
Owner

senny commented Jul 28, 2016

@MattFenelon can't we follow the suggestion in #12 and make HOSTED_VIEWER_ORIGINS configurable?

@MattFenelon
Copy link
Author

Closed by #20

@MattFenelon MattFenelon closed this Aug 3, 2016
@fatuhoku
Copy link

What's... the latest on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants