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

Asciidoctor version update #73

Merged

Conversation

JanWesterkamp-iJUG
Copy link
Contributor

Updates Asciidoctor Maven Plugin and it's dependencies to fix remaining CVE-2022-24823.

It also renamed the existing related version properties to follow Maven conventions, clarification and to align with the MicroProfile (platform) project configuration.
NOTE: I checked usage of this versions in issue #63 before: There is no external use at that time, only internal one in this repo - so it can be changed without breaking something.

This is a fix for the remaining CVE only, extracted from PR #66, that will be updated when this fix is done. MicroProfile (platform) got an similar fix here already and will get a separate PR to align versions there too.

@JanWesterkamp-iJUG
Copy link
Contributor Author

@Emily-Jiang and @radcortez, please review.

Copy link
Member

@Emily-Jiang Emily-Jiang left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you @JanWesterkamp-iJUG !

@Emily-Jiang Emily-Jiang merged commit ef98e49 into microprofile:master Jun 14, 2023
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.

2 participants