Skip to content
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

SIGNING_KEY should be use #301

Open
anhdle14 opened this issue May 14, 2022 · 0 comments
Open

SIGNING_KEY should be use #301

anhdle14 opened this issue May 14, 2022 · 0 comments
Labels
NeedsMoreData Waiting for additional user feedback or case studies

Comments

@anhdle14
Copy link

What happened?

Try to centralize all secrets for config in one place, and found out that signing-key is set up in Helm and not SIGNING_KEY.

The helm values important lines are:

# values.yaml
config:
  existingSigningKeySecret: pomerium-secrets

# pomerium-secrets k8s file
---
kind: Secret
stringData:
  ...
  SIGNING_KEY: ---REDACTED---
  ...

Because helm expects the key to be signing-key

What did you expect to happen?

Use SIGNING_KEY as key and Helm should accept.

Steps to reproduce

  1. Create the pomerium-secret
  2. Apply values.yaml
  3. Error from logs with existing secret does not have signing-key secret key.

What's your environment like?

  • Chart version: irrelevant
  • Container image: irrelevant
  • Kubernetes version: irrelevant
  • Cloud provider: irrelevant
  • Other details: irrelevant

What are your chart values?

...
config:
  existingSigningKeySecret: pomerium-secrets
...

What are the contents of your config secret?

kubectl get secret pomerium -o=jsonpath="{.data['config\.yaml']}" | base64 -D

# irrelevant

What did you see in the logs?

# forgot to record the full logs but it should basically tell the existing secret does not have a key named signing-key.

Additional context

Add any other context about the problem here.

@desimone desimone added the NeedsMoreData Waiting for additional user feedback or case studies label May 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NeedsMoreData Waiting for additional user feedback or case studies
Projects
None yet
Development

No branches or pull requests

2 participants