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

Expand CSPM permissions #16

Merged
merged 2 commits into from
Aug 22, 2024
Merged

Expand CSPM permissions #16

merged 2 commits into from
Aug 22, 2024

Conversation

threepipes
Copy link
Contributor

Why

https://app.cloudbase.ink/docs/release-note/9ae20b1146a443d8a64fb38447b0533f のため

What

  • CSPMの権限について、選定済みのActionを追加
  • Actionが増えたことで、インラインポリシーに収まらなくなったため、カスタマーマネージドポリシーとして作成することに
    カスタマーマネージドポリシーもサイズが上限6,144byteなので、それに収まるように分割した
  • 作成されるカスタマーマネージドポリシーにはユーザー側でPrefixを指定できるようにし、{Prefix}0, {Prefix}1, ... とポリシーが作成されるようにした

@threepipes threepipes self-assigned this Aug 21, 2024
@tetsuuu
Copy link
Contributor

tetsuuu commented Aug 22, 2024

@threepipes
#15 README.md の修正だけなので取り込んじゃってほしいかもですmm

@threepipes threepipes changed the title expand cspm permissions Expand cspm permissions Aug 22, 2024
@threepipes threepipes force-pushed the feature/permission-expantion branch from b329426 to 57dc46c Compare August 22, 2024 05:04
@threepipes threepipes changed the title Expand cspm permissions Expand CSPM permissions Aug 22, 2024
Copy link
Contributor

@MakotoNaruse MakotoNaruse left a comment

Choose a reason for hiding this comment

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

LGTM!

@threepipes threepipes merged commit 9770b53 into main Aug 22, 2024
@tetsuuu tetsuuu deleted the feature/permission-expantion branch August 22, 2024 08:27
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.

3 participants