Skip to content

Conversation

@barkerl
Copy link
Contributor

@barkerl barkerl commented Sep 22, 2025

Description

Re-usable workflow that will rotate a cliend secret for an existing clientID in AWS.

Related issue: JIRA_TICKET_NUMBER

Before submitting (or marking as "ready for review")

  • Does the pull request title follow the conventional commit specification?
  • Have you performed a self-review of the code
  • Have you have added tests that prove the fix or feature is effective and working
  • Did you make sure to update any documentation relating to this change?

Comment on lines +81 to +84
run: |
aws secretsmanager put-secret-value \
--secret-id "${{ inputs.aws-secret-id }}" \
--secret-string "{\"clientId\":\"$CLIENT_ID\",\"clientSecret\":\"$AZURE_NEW_SECRET\"}"
Copy link
Contributor

Choose a reason for hiding this comment

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

The workflow looks good to me, the only query I have is re this last bit.

Would it make sense that AZURE_NEW_SECRET is an output to this workflow instead of updating in place?

Currently, for example, the equivalent CVS secret has more detail that just clientId & clientSecret in the JSON, therefore this would overwrite those extra values.

The other option would be we retrieve the secret as is and merge the two objects together, which will update these keys but also retain the other contents?

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.

2 participants