Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

jetty 10 support #108

Closed
ari opened this issue May 21, 2021 · 7 comments
Closed

jetty 10 support #108

ari opened this issue May 21, 2021 · 7 comments
Milestone

Comments

@ari
Copy link

ari commented May 21, 2021

Some bits of jetty changed and its not possible to just upgrade it without changing this module. Or maybe skip directly to jetty 11 and deal with all the package changes in one go.

@andrus
Copy link
Collaborator

andrus commented May 21, 2021

Good idea. So there are a couple of dimensions to consider:

  • Jetty 10 requires Java 11
  • Jetty 11 requires Java 11 and also switched to Jakarta EE namespace

Now that BQ 2.0 has gone Beta, and we split a new 3.0 line of development, this may be a good time to contemplate our choices. Let me actually start a thread on the Bootique forum, as e.g. a JDK requirements change would affect a lot of modules and a lot of people.

@ari
Copy link
Author

ari commented May 21, 2021

Perhaps there needs to be bootique-jetty11 as a module for people wanting to upgrade. I can't see how it could be clever enough to work out which jetty to depend on depending on what is in the classpath unless there is reflection all over the place.

@andrus
Copy link
Collaborator

andrus commented May 21, 2021

Yeah, that's what we do with other modules (e.g. bootique-cayenne has 3 varieties). It should be explicit.

I am more concerned about conditional JDK dependency, as it would break an implicit assumption that you can mix and match any bootique modules with each other. So 3.0 may have to fully switch to Java 11

@ari
Copy link
Author

ari commented May 21, 2021

I'm not sure what the concern is. The user will figure out pretty quickly that they cannot compile their project.

Why does bootique have to move forward with the most advanced module? If bootique-jetty12 required Java16 next month, would you have to force everyone forward even if they don't install bootique-jetty12?

Right now the move to groovy3 is causing some difficulties between gradle (needs 7) and Cayenne (cgen doesn't work with gradle 7). If bootique has to solve all those sorts of interdependencies its going to be a bumpy ride. Imagine you had some bootique-spock testing module which pulls in groovy and then collides with Cayenne and gradle.

@andrus
Copy link
Collaborator

andrus commented May 21, 2021

Here is the new forum thread on this: https://groups.google.com/g/bootique-user/c/_llobxIJooc

@andrus
Copy link
Collaborator

andrus commented Nov 14, 2021

Now that Jetty 11 is integrated as "bootique-jetty-jakarta", I suppose the default "bootique-jetty" can be switched to Jetty 10. From my experience with 11, it should not be too hard to integrate, and application-side changes will not be required.

andrus added a commit that referenced this issue Feb 6, 2022
@andrus andrus added this to the 3.0.M1 milestone Feb 6, 2022
@andrus
Copy link
Collaborator

andrus commented Feb 6, 2022

bootique-jetty is now on Jetty 10.

@andrus andrus closed this as completed Feb 6, 2022
andrus added a commit that referenced this issue Jul 1, 2022
addressing Jetty bug with DefaultServlet: jetty/jetty.project#8259
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants