-
Notifications
You must be signed in to change notification settings - Fork 157
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
HP Scan invalid Length workaround #215
base: master
Are you sure you want to change the base?
Conversation
Any idea when this might merge? I added it to my local and it seems to be working |
Works for me. Nice solution |
Any idea when they merge into the main branch of this? |
8b6a31c
to
44cb8af
Compare
@boazsegev I've now rebased the pr to current master |
Does this actually solve cases where Wouldn't it be better to test for |
@boazsegev thanks much for looking into it. I've updated my PR:
|
Fixes: #191
It looks that some version of HP Scan produces invalid pdf files. Where the length value would point ot of the file.
I've added a new
relaxed:
option to allow read such files.Not sure if it's any better than #209. More complicated for sure.
Also if we'd need the new option for this?