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

Auto-initialize JAX-RS applications #4481

Merged
merged 2 commits into from
Jun 21, 2023
Merged

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Jun 16, 2023

The pki.spec, pom.xml, and CMake files have been modified to include resteasy-servlet-initializer to enable automatic initialization of JAX-RS applications in all webapps.

The web.xml files in most webapps (except ACME and EST) have been modified to drop the RESTEasy servlet declaration and let the JAX-RS applications be initialized automatically by resteasy-servlet-initializer.

Note: ACME and EST will be updated separately later.

https://docs.jboss.org/resteasy/docs/3.0.24.Final/userguide/html_single/#d4e143

@edewata edewata added the WIP Work In Progress label Jun 17, 2023
@edewata
Copy link
Contributor Author

edewata commented Jun 17, 2023

It's not ready yet due to a build issue.

@edewata edewata force-pushed the resteasy branch 3 times, most recently from f5736cc to b8eedfb Compare June 20, 2023 17:20
The pki.spec, pom.xml, and CMake files have been modified
to include resteasy-servlet-initializer to enable automatic
initialization of JAX-RS applications in all webapps.

https://docs.jboss.org/resteasy/docs/3.0.24.Final/userguide/html_single/#d4e143
The web.xml files in most webapps (except ACME and EST) have
been modified to drop the RESTEasy servlet declaration and
let the JAX-RS applications be initialized automatically by
resteasy-servlet-initializer.

https://docs.jboss.org/resteasy/docs/3.0.24.Final/userguide/html_single/#d4e143
@sonarcloud
Copy link

sonarcloud bot commented Jun 21, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@edewata edewata removed the WIP Work In Progress label Jun 21, 2023
@edewata
Copy link
Contributor Author

edewata commented Jun 21, 2023

This is ready now.

Copy link
Contributor

@ckelleyRH ckelleyRH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@fmarco76 fmarco76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@edewata
Copy link
Contributor Author

edewata commented Jun 21, 2023

@ckelleyRH @fmarco76 Thanks!

@edewata edewata merged commit ed6b8c6 into dogtagpki:master Jun 21, 2023
134 of 137 checks passed
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