Skip to content
This repository has been archived by the owner on Feb 9, 2024. It is now read-only.

Switch from IText to OpenPDF fork #66

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

Conversation

mandrean
Copy link

@mandrean mandrean commented Jun 21, 2019

ghost4j uses a ~10 year old dependency of itext with a kinda corrupt dependency declaration on some bouncy castle stuff in its POM file. (It should have been org.bouncycastle:bctsp-jdk14:1.38 instead of bouncycastle:bctsp-jdk14:138 etc.)

This PR bumps the IText dependency (which by the way was renamed to IText PDF and the reverse domain was changed from com.lowagi to com.itextpd).

EDIT Oct-13-2020: Updated PR to use LibrePDF/OpenPDF LGPL fork of IText 4 instead.

Closes #54.

@MartinWillkomm
Copy link

AFAIK 2.1.7 is the last iText version under LGPL License (and not AGPL) and the last really "free"version.
It changed after that. If you increase the dependency, you might be introducing licensing problems with other project/programs. see here: https://stackoverflow.com/questions/4321207/what-is-latest-version-of-itext-that-is-not-agpl

@mandrean
Copy link
Author

@4F2E4A2E
Copy link
Collaborator

@mandrean thanks for this MR, but we can not afford AGPL as you can imagine.
Is there any other alternative?

@mandrean mandrean changed the title Bump IText dependency version Switch from IText to OpenPDF fork Oct 13, 2020
@mandrean
Copy link
Author

mandrean commented Oct 13, 2020

@MartinWillkomm @4F2E4A2E ~1.5 years later, I have an alternative solution:

There is an actively maintained LGPL fork of IText 4 called LibrePDF/OpenPDF.

And it uses the old package namespaces, so it should work as a drop-in replacement in our case: https://github.com/LibrePDF/OpenPDF/blob/master/pdf-toolbox/src/test/java/com/lowagie/examples/general/HelloWorldPdf.java#L21

I updated this PR to use that instead. 🚀

@mandrean
Copy link
Author

Ping @MartinWillkomm @4F2E4A2E

@mandrean
Copy link
Author

Ping @4F2E4A2E @MartinWillkomm

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

Successfully merging this pull request may close these issues.

iText dependency is quite outdated. Any plans to upgrade it?
3 participants