-
Notifications
You must be signed in to change notification settings - Fork 4
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
As ACF tech lead, I need all secret keys rotated and rotation steps updated #1493
Comments
Discuss the priority of this alongside the update to where to store secret key markdown. Talk about at next tech sync on Friday 1.7. |
@ADPennington and @abottoms-coder to sync on this issue at next tech sync to pull into next sprint. Per backlog refinement 1.25.22 |
This ticket is still required for epic #972 but it does not have have to be completed as part of v1.0. It can be done as part of a supplemental release. |
Likely waiting for NexGen XMS to complete. |
this will probably be closed but will revisit after decision is made on login.gov vs nextgen. #1818 is updating this for login.gov. |
actually, nevermind the above note 😄 We have at least 3 sets of secret keys that we need to ensure are rotated quarterly:
This issue's description has been updated to capture goals to document the rotation steps for these keys |
We decided to close out the secret key mitigation epic for release 1 after manual steps for rotating secret keys are centrally documented. In a future release we will revisit discussion for a more automated solution for rotating these keys.
Currently, there are only a couple of keys that require rotation on quarterly basis:
jwt key
for login.gov-- the rotation steps are documented here. These steps should be reviewed, updated if needed, and followed.django secret key
- the rotation steps are not yet documented in the repoams secret key
for ACF AMS - the rotation steps are not yet documented in the repoSince there are only a couple, and in an effort to keep the number of markdowns down to what is necessary, we can update the existing markdown to be inclusive of all these keys
ACs:
Tasks:
Notes:
The text was updated successfully, but these errors were encountered: