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

Move the check for encryption into PdfParser instead of the CrossReference class #21

Open
JanSlabon opened this issue Sep 13, 2017 · 2 comments
Milestone

Comments

@JanSlabon
Copy link
Member

To place the logic in the CrossReference class looks somewhat odd. We should refactor this.

@JanSlabon
Copy link
Member Author

And throw a more context specific exception: e.g. UnsupportedException.

@JanSlabon JanSlabon added this to the 3 milestone Sep 11, 2018
@JanSlabon
Copy link
Member Author

The move is actually impossible because we need to check for encryption when we reach a compressed cross reference. If we would not check for this a missleading exception will be thrown, that would say that the file can be processed with the FPDI PDF-Parser add-on, which is not true.

The new exception type is postponed to v3.

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

No branches or pull requests

1 participant