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

Create multi-module project #348

Closed
lhazlewood opened this issue Jul 11, 2018 · 1 comment
Closed

Create multi-module project #348

lhazlewood opened this issue Jul 11, 2018 · 1 comment
Assignees
Milestone

Comments

@lhazlewood
Copy link
Contributor

JJWT 0.10.0 will support different JSON and Base64 implementations and allow users to plug-in their own implementations where necessary.

As a result, all 3rd-party libraries should be moved to individual jjwt modules so users can depend on only what they need and not pull in more classes or dependencies than are required.

This issue is to track the work to:

  • Change the existing pom.xml to reflect a jjwt-root pom artifact
  • split existing code into a number of modules:
    • jjwt-api: contains classes and interfaces users will add as a compile-time dependency to their project
    • jjwt-impl: contains classes and interfaces users will add as a runtime dependency in their project. This artifact will contain private implementation-only classes and interfaces that users should never depend on in a project and where semver is not guaranteed and can change at any time.
    • jjwt-jackson: contains Jackson implementation for JSON (de)serialization. Only needed as a dependency if you use Jackson in your project
    • jjwt-orgjson: contains org.json implementations for JSON (de)serialization. Only needed as a dependency if you want to use org.json in your project. Android users will want this since it is natively supported since Android version 1 and does not require any additional 3rd party dependencies.
@lhazlewood lhazlewood added this to the 0.10.0 milestone Jul 11, 2018
@lhazlewood lhazlewood self-assigned this Jul 11, 2018
This was referenced Jul 11, 2018
@lhazlewood
Copy link
Contributor Author

Completed via #351 - merged to master, will be released for 0.10.0

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

No branches or pull requests

1 participant