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

Add jaxrs-server-security to catalog #6757

Merged
merged 1 commit into from Jan 17, 2022

Conversation

pgressa
Copy link
Contributor

@pgressa pgressa commented Jan 14, 2022

@pgressa pgressa changed the title Add jaxrs-server-security-to-catalog to catalog Add jaxrs-server-security to catalog Jan 14, 2022
@pgressa pgressa requested a review from melix January 14, 2022 10:03
@graemerocher
Copy link
Contributor

Ideally in the future the module should publish a BOM

@pgressa
Copy link
Contributor Author

pgressa commented Jan 14, 2022

Maybe we should have automatically add bom support to all projects while working on this new build plugins. But for now, the launcher with jaxrs-security just fails since it doesn't have proper version set.

@melix
Copy link
Contributor

melix commented Jan 17, 2022

I'm not sure adding a BOM to all modules makes sense. It probably only makes sense for multi-module projects. Note that we can also improve our BOMs for Gradle, to enable version alignment.

@pgressa
Copy link
Contributor Author

pgressa commented Jan 17, 2022

I'm not sure adding a BOM to all modules makes sense. It probably only makes sense for multi-module projects. Note that we can also improve our BOMs for Gradle, to enable version alignment.

Yes, I was referring to multi-module projects only.

@pgressa pgressa merged commit e5f03a5 into 3.2.x Jan 17, 2022
@pgressa pgressa deleted the pgressa-jaxrs-server-security-to-catalog branch January 17, 2022 10:45
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.

None yet

3 participants