iann0036 / aws.permissions.cloud

A crowdsourced AWS IAM permissions reference.
https://aws.permissions.cloud
MIT License
84 stars 9 forks source link

feat: Search managed policy from IAM action #8

Closed exoego closed 9 months ago

exoego commented 10 months ago

Closes https://github.com/iann0036/aws.permissions.cloud/issues/6

Depends on https://github.com/iann0036/iam-dataset/pull/31

1. Search managed policies by action

The search modal can show "Managed Policies" that contains the search term (IAM action).

image

I've dropped https://github.com/iann0036/aws.permissions.cloud/pull/8/commits/06cc35b2ca803efac5e43b66df12174c1dcd2146 and https://github.com/iann0036/aws.permissions.cloud/pull/8/commits/116c4d318ac9c727f1612a98ed6cb23f4d9e5589 since those are controversial.

dropped from this PR ## 2. Quick-search by clicking action IAM action names are now linked to search. When user clicks the action name, ![image](https://github.com/iann0036/aws.permissions.cloud/assets/127635/342f4c0b-798c-49b6-a6c2-5bae1460d73b) It opens a search modal showing the matched managed policies. ![image](https://github.com/iann0036/aws.permissions.cloud/assets/127635/7458c905-0d0d-45fd-8868-5dfe4464ea57) ## 3. Dedicated page for Managed Policies for Service Each service now has a third page: Managed Policies for Service. The new page shows the list of managed policies that contain IAM Actions for the service. ![image](https://github.com/iann0036/aws.permissions.cloud/assets/127635/8a68ab35-aa54-44b9-a2ef-2ba54cc206a3) As a consequence, I've tweaked buttons/tabs as described below: ![image](https://github.com/iann0036/aws.permissions.cloud/assets/127635/45faa8d1-fd17-4b69-a866-352e3a292c1e)
iann0036 commented 10 months ago

Do you have an idea on how we'd represent this data? Maybe we could have a dedicated page view for an IAM action's detail, however I would like to keep the main list view as the primary reference where possible. 🤔

exoego commented 10 months ago

Maybe we could have a dedicated page view for an IAM action

That was my first thought, but a new dedicated page requires much effort. I will consider a more minimal approach.


After consideration, I think it is better to have 3rd page to show Managed Policies for each Service.

iann0036 commented 9 months ago

Hey @exoego,

Still analysing this one. I'm still pondering how to support a managed policies for action view, however I'm unlikely to accept a managed services for service view as it has some weird consequences and services are somewhat more vague than pure actions.

exoego commented 9 months ago

@exoego I've excluded

iann0036 commented 9 months ago

Thanks @exoego,

I'll merge this one now. If you could keep your branch around whilst we figure out the best way to include the view, that'd be great. We'll use the majority of it...eventually.

iann0036 commented 9 months ago

@exoego I'm thinking about something like:

Screenshot 2023-10-07 at 2 08 47 pm