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

[Known issue][Detection Engine] Exception item comment validation on newline chars #6275

Closed
Tracked by #6222
yctercero opened this issue Dec 5, 2024 · 10 comments
Closed
Tracked by #6222

Comments

@yctercero
Copy link
Contributor

Summary

I wasn't sure if this required a docs issue as it would get picked up by release notes, but will create just in case.

Issue: elastic/kibana#201820
PR: elastic/kibana#202063
Workaround: elastic/kibana#201820 (comment)

Fix slated for 8.16.2

Bug Summary

After upgrading a deployment from any version below 8.16.0 to 8.16+, any rule exceptions with comments containing newline char \n will throw a 500 in the exception items _find api, like when trying to view the exceptions in the rule details page.

@nastasha-solomon
Copy link
Contributor

This was added to the Serverless release notes via elastic/docs-content#235.

@nastasha-solomon
Copy link
Contributor

Bug doc'd in the known issue sections for 8.16.0 and 8.16.1:
https://www.elastic.co/guide/en/security/8.16/release-notes-header-8.16.0.html

@nicpenning
Copy link

@yctercero
Copy link
Contributor Author

@nicpenning Please find the backport to 8.17 here - elastic/kibana#203708

@yctercero
Copy link
Contributor Author

@dhurley14 could you confirm if it got into 8.17 or 8.17.1?

@nicpenning
Copy link

The known issue exists is 8.17, but is not shown as known issue in 8.17.1 which I believe is correct, however, there is no mention of the fix in 8.17.1.

@dhurley14
Copy link
Contributor

The fix was skipped for 8.17.0 and added 8.17.1.

@dhurley14
Copy link
Contributor

Kibana machine added the 8.17.0 label, even though my PR I manually added 8.17.1 as that was the target release. We wanted to skip 8.17.0 since it was a day or two before the final BC.

elastic/kibana#202063 (comment)

@nastasha-solomon
Copy link
Contributor

Thanks, all. I'll open a PR today to update the 8.17.1 release notes.

@nastasha-solomon
Copy link
Contributor

8.17.0 and 8.17.1 release notes are updated: https://www.elastic.co/guide/en/security/8.17/release-notes-header-8.17.0.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants