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

Cherry-pick #17944 to 7.7: corrected module decode_cef documentation from Beta to GA #18176

Merged
merged 2 commits into from
May 12, 2020

Conversation

andrewkroh
Copy link
Member

@andrewkroh andrewkroh commented May 4, 2020

Cherry-pick of PR #17944 to 7.7 branch. Original message:

What does this PR do?

decode_cef codec is not beta any more.
For the moment this is not consistent with https://www.elastic.co/guide/en/beats/filebeat/current/filebeat-module-cef.html

Why is it important?

https://www.elastic.co/guide/en/beats/filebeat/current/processor-decode-cef.html and https://www.elastic.co/guide/en/beats/filebeat/current/filebeat-module-cef.html have differences regarding documentation. This should be fixed after this PR is merged.

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label May 4, 2020
decode_syslog is no longer in beta. It is used by the cef module which is not beta.

Co-authored-by: Andrew Kroh <andrew.kroh@elastic.co>
(cherry picked from commit 2e4c7c3)
@andrewkroh andrewkroh added Team:SIEM and removed needs_team Indicates that the issue/PR needs a Team:* label labels May 5, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/siem (Team:SIEM)

Copy link
Contributor

@ycombinator ycombinator left a comment

Choose a reason for hiding this comment

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

Backport LGTM (but there's a conflict on the CHANGELOG)

@andrewkroh andrewkroh merged commit d369c45 into elastic:7.7 May 12, 2020
@andrewkroh andrewkroh deleted the backport_17944_7.7 branch January 14, 2022 14:06
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
decode_syslog is no longer in beta. It is used by the cef module which is not beta.

Co-authored-by: Andrew Kroh <andrew.kroh@elastic.co>
(cherry picked from commit b65a342)

Co-authored-by: André Letterer <48132449+111andre111@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants