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

Migrate from iText to PDFBox #2652

Closed
mbastian opened this issue Oct 5, 2022 · 0 comments · Fixed by #2657
Closed

Migrate from iText to PDFBox #2652

mbastian opened this issue Oct 5, 2022 · 0 comments · Fixed by #2657
Assignees
Milestone

Comments

@mbastian
Copy link
Member

mbastian commented Oct 5, 2022

The PDF library we use (iText) is under AGPL license only. We know that this license is problematic for some people and companies so we would like to replace this dependency with another one.

Proposed solution

PDFBox

Alternatives considered

PDFBox seems like the best alternative. Equivalent in features we need and with Apache license.

Additional context

Based on #2361 iText is the only AGPL dependency we have.

@mbastian mbastian added this to the 0.10.0 milestone Oct 5, 2022
@mbastian mbastian self-assigned this Oct 5, 2022
@mbastian mbastian linked a pull request Oct 6, 2022 that will close this issue
8 tasks
mbastian added a commit that referenced this issue Oct 8, 2022
* Add library to CoreLibraryWrapper

* Migrate UI to use PDFBox

* Migrate node renderer

* Migrate edge and arrow renderer

* Migrate PDFTarget with font support

* Migrate PDFExporter

* Migrate node label renderer

* Fix issue when testing with labels

* Add PDFUtils

* Migrate edge label renderer

* Fix font loading issue

* Make export succeed in case of exception

* Remove embedded font

* Fix edge color

* Fix landscape mode

* Update API changes

* Formatting

* Refactor

* Remove dependency on iText

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

Successfully merging a pull request may close this issue.

1 participant