Skip to content

Conversation

jamesiarmes
Copy link
Member

Add missing parameters and use conditions to properly scope permissions where possible.

@jamesiarmes jamesiarmes requested a review from a team as a code owner October 6, 2025 16:54
@jamesiarmes jamesiarmes requested a review from Copilot October 6, 2025 17:37
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR cleans up IAM permissions required for deployment by reorganizing and properly scoping permissions using resource constraints and conditions. The main focus is on improving security by limiting permissions to specific resources where possible.

Key changes include:

  • Reorganized permissions into logical groups with descriptive Sids
  • Added resource-level constraints to limit permissions to project-specific resources
  • Implemented conditions to further scope permissions based on tags and request context

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jamesiarmes jamesiarmes merged commit 10082ca into main Oct 6, 2025
9 checks passed
@jamesiarmes jamesiarmes deleted the deployment-permissions branch October 6, 2025 17:52
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