Skip to content

Conversation

@ywangd
Copy link
Member

@ywangd ywangd commented Dec 4, 2020

This is a follow-up PR for #65256 to fix the xpack info and usage reports for operator privilegs. In summary, this PR ensures:

  • _xpack does not report operator privileges because it is categorised under security
  • _xpack/usage reports operator privileges status under the security section
  • _license/feature_usage reports last used time of operator privileges. It is up to the downstream to filter out this report if necessary.

I tag this PR as non-issue because the original changes are not released yet. Also this will not be backported separately because it will be backported along with the main changes (#65256).

@ywangd ywangd added >non-issue :Security/Security Security issues without another label v8.0.0 labels Dec 4, 2020
@ywangd ywangd requested a review from tvernum December 4, 2020 02:41
@elasticmachine elasticmachine added the Team:Security Meta label for security team label Dec 4, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

@ywangd
Copy link
Member Author

ywangd commented Dec 4, 2020

@elasticmachine update branch

@tvernum
Copy link
Contributor

tvernum commented Dec 4, 2020

It is up to the downstream to filter out this report if necessary.

I assume you will make sure they know that.

Copy link
Contributor

@tvernum tvernum left a comment

Choose a reason for hiding this comment

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

LGTM

@ywangd
Copy link
Member Author

ywangd commented Dec 4, 2020

It is up to the downstream to filter out this report if necessary.

I assume you will make sure they know that.

Good point. I will initiate an email for it.

@ywangd ywangd merged commit 201b25e into elastic:master Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>non-issue :Security/Security Security issues without another label Team:Security Meta label for security team v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants