ParseSecrets incorrectly matching secrets on deploy ("secret not found") #297
Labels
No Label
abra
abra-gandi
awaiting-feedback
backups
bug
build
ci/cd
community organising
contributing
coopcloud.tech
democracy
design
documentation
duplicate
enhancement
finance
funding
good first issue
help wanted
installer
kadabra
performance
proposal
question
recipes.coopcloud.tech
security
test
wontfix
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: coop-cloud/organising#297
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Describe the current behavior
I realised that it is
length=64
in the.env.sample
of the outline recipe and that there is a bug inParseSecrets
that is doing matching onsecret # foo
instead ofsecret
.Steps to reproduce
Set
length=555555
(anything that doesn't match the.env.sample
).Describe the expected behavior
We parse stuff correctly.
Any idea how this might be fixed?
It's kind of unclear, I think I need to abstrat the secret parsing logic out of
pkg/secret/...
so it can be shared inupstream/...
. It's a bit scary because I'm not sure what this will break.Work around is just set your
length=...
to what is set in the recipe.env.sample
OR simply delete the length specifier. This has helped me re-deploy existing recipes. I haven't tested from scratch in a while but I will.