Skip to content

change use oicd for ga to aws#40

Merged
limit7412 merged 4 commits into
masterfrom
develop
Mar 31, 2026
Merged

change use oicd for ga to aws#40
limit7412 merged 4 commits into
masterfrom
develop

Conversation

@limit7412
Copy link
Copy Markdown
Owner

@limit7412 limit7412 commented Mar 31, 2026


Open with Devin

Note

Medium Risk
Changes CI deployment authentication, so misconfigured OIDC/role trust could break PR deployments; otherwise the change reduces secret exposure by removing static AWS keys.

Overview
Updates the serverless-dev GitHub Actions workflow to use GitHub OIDC for AWS authentication by granting id-token: write and switching configure-aws-credentials from long-lived access keys to role-to-assume (secrets.AWS_ROLE_ARN).

Written by Cursor Bugbot for commit e1975ff. This will update automatically on new commits. Configure here.

@gemini-code-assist
Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

cursor[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

cursor[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@cursor
Copy link
Copy Markdown

cursor Bot commented Mar 31, 2026

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@cursor
Copy link
Copy Markdown

cursor Bot commented Mar 31, 2026

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@limit7412
Copy link
Copy Markdown
Owner Author

該当エラーに関してメインブランチにマージされる必要があるとの情報があったので一度マージして動作確認する

@limit7412 limit7412 merged commit 9c920cb into master Mar 31, 2026
1 of 5 checks passed
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