-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[blog] CVE-2024-36129 post followup #4605
[blog] CVE-2024-36129 post followup #4605
Conversation
linkTitle: Security vulnerability in confighttp and configgrpc | ||
Security vulnerability in OTel Collector confighttp and configgrpc | ||
(CVE-2024-36129) | ||
linkTitle: Collector vulnerability CVE-2024-36129 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are other alternative short titles (which show up in the left nav), one of which is just the CVE ID.
Also, I wasn't sure if you felt that "Security" had to be part of the short title or not. Anyhow, let me know what you think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No strong feelings about whether to put 'Security' in there or not, I am fine with removing if that makes things easier
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As long as the title says configgrpc and confighttp, the link title can omit them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to merge as is from my side
linkTitle: Security vulnerability in confighttp and configgrpc | ||
Security vulnerability in OTel Collector confighttp and configgrpc | ||
(CVE-2024-36129) | ||
linkTitle: Collector vulnerability CVE-2024-36129 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No strong feelings about whether to put 'Security' in there or not, I am fine with removing if that makes things easier
3e53065
to
08525fe
Compare
Co-authored-by: Severin Neumann <neumanns@cisco.com>
linkTitle
/cc @jpkrohling @mx-psi @arminru