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

Ignore errors when parsing optional array of decode parameter dicts #860

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cjpartridgeb
Copy link

Fixes #859

  • No longer error when failing to parse optional decode param dicts for pdf filter pipeline
  • No longer try to validate above dict when its failed to parse

As noted in the issue, please review the changes I've made as I'm no Go expert, nor PDF expert.

* No longer error when failing to decode optional dicts for pdf filter pipeline
* No longer try to validate above dict when its failed to parse
@CLAassistant
Copy link

CLAassistant commented Apr 25, 2024

CLA assistant check
All committers have signed the CLA.

@hhrutter
Copy link
Collaborator

hhrutter commented Apr 25, 2024

Ignoring DecodeParms of an unexpected type in read.go should probably be sufficient in your situation
but I have to insist on a testfile going along with this patch.

@RevealedSoulEven
Copy link

This was really needed. Thanks. Can you please share compiled executables please

@RevealedSoulEven
Copy link

#384

Does this also ignore these errors?

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.

Dereference error with a particular (corrupt?) PDF
4 participants