fusionauth-jwt icon indicating copy to clipboard operation
fusionauth-jwt copied to clipboard

Upgrade to Java >= 14?

Open robotdan opened this issue 3 years ago • 3 comments

How many people would I break if I moved this to Java >= 14?

Feel free to comment on this thread.

Related

  • https://github.com/FusionAuth/fusionauth-jwt/issues/13#issuecomment-1168946774

robotdan avatar Apr 22 '21 15:04 robotdan

Fine by me.

Yaytay avatar May 26 '22 06:05 Yaytay

Thanks for the feedback @Yaytay. If we update the min. required version of Java, I would likely select an LTS, version 11 or 17.

robotdan avatar May 27 '22 16:05 robotdan

I was just looking at trying to use Edwards Curve and it is only supported in Java 15 or greater

https://openjdk.org/jeps/339

I have contemplated forking this project to get it since the Algorithm is an enum.

agentgt avatar Jun 28 '22 16:06 agentgt

Should probably look at java 17 now, since it is an LTS: https://www.oracle.com/news/announcement/oracle-releases-java-17-2021-09-14/

mooreds avatar Jan 19 '23 14:01 mooreds

There is some work in progress to make this library more modular so the base can optionally stay on Java 8, and we can add support for various new algorithms using extensions.

https://github.com/FusionAuth/fusionauth-jwt/issues/13#issuecomment-1229122878

robotdan avatar Feb 09 '23 04:02 robotdan

LTS versions 8, 11, 17, 21. https://www.oracle.com/java/technologies/java-se-support-roadmap.html

Probably safe to ditch Java 8 since support is only available through extended or sustaining support from Oracle.

robotdan avatar Aug 25 '23 18:08 robotdan

Planning to leave the base library at Java 8 for compatibility. I don't know that there is any technical benefit to using something later.

We could move to Java 11, to pick up a more recent LTS, but I don't know that it matters much. We should be able to release support for EDDSA w/out requiring the base library use >= Java 17.

robotdan avatar Aug 25 '23 23:08 robotdan