This looks like a tiny and valuable PR for securit...
# atlantis-contributorsr
RB
09/06/2024, 3:16 PMThis looks like a tiny and valuable PR for security. This should remove the token from the logs.
https://github.com/runatlantis/atlantis/pull/4667
merged1 1
RB
09/06/2024, 3:16 PMI approved. Just thought I’d reach out if anyone has any reservations on merging
p
PePe Amengual
09/06/2024, 3:17 PMtotally, we are pretty far behind on reviews
PePe Amengual
09/06/2024, 3:17 PMsorry
r
RB
09/06/2024, 3:18 PMIt’s all good. we all get busy 🙂
👍🏽 1
RB
09/08/2024, 3:16 PMSince this is a security fix, can we get it cherrypicked into a new patch release on maintained minor versions (assuming 0.29, 0.28, and 0.27) ?
p
PePe Amengual
09/08/2024, 3:48 PMYes
PePe Amengual
09/08/2024, 4:35 PMwe should
PePe Amengual
09/08/2024, 4:35 PMdo you want to create the cherry pick PRs?
r
RB
09/08/2024, 5:21 PMI don't know how 😬
p
PePe Amengual
09/08/2024, 6:22 PMyou have to.manucl cherry pick to.the branch
PePe Amengual
09/08/2024, 6:23 PMthe bot is a bit useless