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
citrix_waf: relax header format constraints #9285
Conversation
🚀 Benchmarks reportPackage
|
Data stream | Previous EPS | New EPS | Diff (%) | Result |
---|---|---|---|---|
log |
1396.65 | 1161.44 | -235.21 (-16.84%) | 💔 |
To see the full report comment with /test benchmark fullreport
d9268c7
to
f3eb312
Compare
- allow missing citrix header - allow syslog priority field - allow year-first date format in timestamp
💚 Build Succeeded
History
cc @efd6 |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Pinging @elastic/sec-deployment-and-devices (Team:Security-Deployment and Devices) |
@jamiehynds The owner label here disagrees with the label on the issue. Which is correct? |
Given that Citrix WAF is an appliance that syslogs I think it lives with deployment and devices, but will let @norrietaylor or @narph confirm. |
the team owner was probably wrongly assigned in #9266 but indeed Citrix WAF is currently owned by deployment and devices team. |
Agreed @narph! Thanks for getting here first. |
Pinging @elastic/sec-deployment-and-devices (Team:Security-Deployment and Devices) |
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.
LGTM
Package citrix_waf - 1.15.0 containing this change is available at https://epr.elastic.co/search?package=citrix_waf |
Proposed commit message
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots