Closed zenmonkeykstop closed 3 years ago
LGTM, just blocking on https://github.com/freedomofpress/k8s-configs/pull/205 being merged.
I don't mind, but If it bothers you at all one of my commits is duplicated with [TEST]
in the commit message (probably from being cherry-picked) and it could be rebased out.
I'm fine with it, given the nature of the repo I'd rather preserve the history intact.
Tagging myself in on review here. Will coordinate actual merge (and therefore deploy) with @maeve-fpf ahead of time. First, I'll review the changes locally.
Comments addressed, @maeve-fpf. I'm going to push this to staging, as you advised, and if no problems, will proceed with prod deploy later today.
Status
Ready for review
This PR pushes the historical changes in the
key-rotation-2021
branch tomain
, updating the current ruleset to the latest one signed with the 2021 release key.Review Checklist
Path Prefix
:https://raw.githubusercontent.com/freedomofpress/securedrop-https-everywhere-ruleset/$BRANCH_NAME
Post-Deployment Checklist
main
branch, instead of fromkey-rotation-2021
.