Skip to content

Conversation

@luxas
Copy link

@luxas luxas commented Aug 23, 2024

Hi 👋 ! Thanks a lot for this project!

Here's a couple of improvements (at least in my opinion, let me know what you think 😄) to the project:

  • Text output in resource view is now sorted by API group, so it's easier for a human reviewer of the output to look at similar resources together. Also it is easier to read what the name of the resource is when the api group name is not duplicated for every resource.
  • Previously, subresources like /status and /scale, or importantly serviceaccounts/token or certificatesigningrequests/approval, were not listed. By looking up all resources within a groupversion, we now include this data in the output.
  • API group was not taken into account in the subject view, which means that if there was an RBAC rule that gave a user access to deployments.mycustomgroup, it would look like that user also had access to deployments.apps

Let me know what you think about these!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant