Skip to content

Conversation

@sotty
Copy link
Contributor

@sotty sotty commented Dec 20, 2021

@ranma2913
Copy link

pom.xml Outdated
<!-- CVE-2021-45046: https://nvd.nist.gov/vuln/detail/CVE-2021-45046 -->
<log4j2.version>2.17.0</log4j2.version>

<!-- https://jira.qos.ch/browse/LOGBACK-1591 -->

Choose a reason for hiding this comment

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

Suggested change
<!-- https://jira.qos.ch/browse/LOGBACK-1591 -->
<!-- CVE-2021-42550: https://nvd.nist.gov/vuln/detail/CVE-2021-42550 -->

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thank you - I have updated the link

Choose a reason for hiding this comment

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

Great. Thanks

pom.xml Outdated
<!-- CVE-2021-45046: https://nvd.nist.gov/vuln/detail/CVE-2021-45046 -->
<log4j2.version>2.17.0</log4j2.version>

<!-- https://jira.qos.ch/browse/LOGBACK-1591 -->

Choose a reason for hiding this comment

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

Great. Thanks

@sotty
Copy link
Contributor Author

sotty commented Dec 21, 2021

@ranma2913 @fantavlik Are there any next steps, or concerns, on getting this PR approved and merged?
Thank you in advance

@sotty
Copy link
Contributor Author

sotty commented Dec 22, 2021

I have found and signed the contributor agreement - should I also open an Issue for this PR?

@sotty
Copy link
Contributor Author

sotty commented Dec 22, 2021

Rebased + squashed to keep the history linear

@fantavlik fantavlik merged commit 9217bef into splunk:main Dec 23, 2021
@fantavlik
Copy link
Contributor

Thanks @sotty! This is out as v1.11.4 now.

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