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

veraPDF does not run on EBCDIC-Platforms like z/OS #1032

Closed
michaelknigge opened this issue Jul 12, 2019 · 4 comments
Closed

veraPDF does not run on EBCDIC-Platforms like z/OS #1032

michaelknigge opened this issue Jul 12, 2019 · 4 comments
Assignees
Labels
bug A product defect that needs fixing P1 High priority issues to be scheduled in the upcoming release

Comments

@michaelknigge
Copy link

We've tried to use veraPDF for PDF/A validation on z/OS - but sadly in vain. This is because veraPDF assumes that it is running on an ASCII based platform and for this reason does not provide a Charset on various byte[] to String and/or String to byte[] transformations.

I did some minor modifications and got it working. I'd like to create a pull request for this if there is any chance it gets integrated.

The only part that needs to be modified is veraPDF-parser.

Could you please assist me?

  1. for what branch I should create the pull request (I guess "integration")?
  2. Any chance it gets integrated?
  3. if yes, can you give me a timeframe when it might be integrated and when a new version (with the fix) gets released (we need it at a customer site soon)?

Thanks & bye,
Michael

@carlwilson
Copy link
Contributor

Hi @michaelknigge thanks for your input, I'll answer your questions as a list:

  1. Yes the integration branch is the one to PR against please.
  2. I'll review the PR and comment but in principle it sounds a useful addition, particularly if we're making encoding assumptions.
  3. If yes then in the 1.16 release in early 2020.

If you set up the PR and ping me I'll take a look within a couple of days.

Thanks,
Carl

@michaelknigge
Copy link
Author

I've created a pull request - see veraPDF/veraPDF-parser#377

@BezrukovM
Copy link
Contributor

@michaelknigge Thanks for your issue and PR. This fix will also required to make a release hotfix about COSName utf-8 validation, so I added all your changes into the another PR against the release branch. The issue will be closed as soon as PR with fix will be merged.

@ghost ghost added bug A product defect that needs fixing P1 High priority issues to be scheduled in the upcoming release labels Oct 24, 2019
@ghost ghost assigned BezrukovM Oct 24, 2019
@BezrukovM
Copy link
Contributor

Merged into release and dev.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A product defect that needs fixing P1 High priority issues to be scheduled in the upcoming release
Projects
None yet
Development

No branches or pull requests

3 participants