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.

Sign up for GitHub

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

Jump to bottom

[Security Solution] [Detections] Bug fix for read privilege check during rule exec #86852

Merged
merged 1 commit into from
Dec 23, 2020
Merged

[Security Solution] [Detections] Bug fix for read privilege check during rule exec #86852

merged 1 commit into from
Dec 23, 2020

Conversation

dhurley14
Copy link
Contributor

Summary

Logic bug for determining when to write the partial failure status.

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@dhurley14 dhurley14 self-assigned this Dec 22, 2020
@dhurley14 dhurley14 added release_note:skip Skip the PR/issue when compiling release notes review Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Detections and Resp Security Detection Response Team v7.11.0 v7.12.0 v8.0.0 labels Dec 22, 2020
@dhurley14 dhurley14 marked this pull request as ready for review December 22, 2020 22:54
@dhurley14 dhurley14 requested review from a team as code owners December 22, 2020 22:54
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

yctercero
yctercero approved these changes Dec 23, 2020
View reviewed changes
Copy link
Contributor

@yctercero yctercero left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the fix.

@dhurley14 dhurley14 merged commit 35b10b5 into elastic:master Dec 23, 2020
@dhurley14 dhurley14 deleted the hotfix-read-priv-check branch December 23, 2020 02:14
This was referenced Dec 23, 2020
Merged
Merged
dhurley14 added a commit to dhurley14/kibana that referenced this pull request Dec 23, 2020
@dhurley14
[Security Solution] [Detections] Bug fix for read privilege check dur…
ffd8a10 
…ing rule exec (elastic#86852)
dhurley14 added a commit to dhurley14/kibana that referenced this pull request Dec 23, 2020
@dhurley14
[Security Solution] [Detections] Bug fix for read privilege check dur…
8d573a8 
…ing rule exec (elastic#86852)
dhurley14 added a commit that referenced this pull request Dec 23, 2020
@dhurley14
[7.11] [Security Solution] [Detections] Bug fix for read privilege ch…
ea89d75 
…eck during rule exec (#86852) (#86859)
dhurley14 added a commit that referenced this pull request Dec 23, 2020
@dhurley14
[7.x] [Security Solution] [Detections] Bug fix for read privilege che…
9f202aa 
…ck during rule exec (#86852) (#86858)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yctercero yctercero yctercero approved these changes

Assignees

@dhurley14 dhurley14

Labels
release_note:skip Skip the PR/issue when compiling release notes review Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v7.11.0 v7.12.0 v8.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dhurley14 @kibanamachine @yctercero