Skip to content

Conversation

@waynedyck
Copy link

This pull request also incorporates the updates in pull request #224 from @madoxas.

With the shift from javax.* to jakarta.* package names, the minimum supported Tomcat version is 10.1 at the moment and it uses servlet version 6.0. I also added the web.xml servlet definition for Tomcat version 11.

@weavejester
Copy link
Owner

Thanks for the PR. Can you ensure that the subject line of the commit messages is 50 characters or less, and the body of the commit wraps at 72 characters?

These are required for Tomcat versions 10.1 and 11.

The default-servlet-version is bumped to "6.0" as Tomcat 10.1 is the
first version utilizing the shift from javax.* to jakarta.* package
names.
@waynedyck
Copy link
Author

Thanks for the PR. Can you ensure that the subject line of the commit messages is 50 characters or less, and the body of the commit wraps at 72 characters?

Done.

@weavejester
Copy link
Owner

Perfect. And just to confirm - have you tested this change? Does it work with Tomcat?

@waynedyck
Copy link
Author

Perfect. And just to confirm - have you tested this change? Does it work with Tomcat?

Yes. I just tested the changes again on an application with lein ring uberwar and was able to deploy the .war file and have it run on Tomcat 11.0.10 without issues.

The current version, [lein-ring "0.12.6"] still works on Tomcat 9.0.73 without the changes.

@waynedyck
Copy link
Author

@weavejester Do you require anything else from me at this point regarding the pull request?

jeffdik pushed a commit to jeffdik/lein-ring that referenced this pull request Aug 29, 2025
To have a deployed version of [Update ring-jakarta-servlet dependency version and add web.xml for versions 6.0 and 6.1 by waynedyck · Pull Request weavejester#225 · weavejester/lein-ring · GitHub](weavejester#225)
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

Successfully merging this pull request may close these issues.

3 participants