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

feat: Add authorization overlays #5695

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

anticorrelator
Copy link
Contributor

resolves #4516

Add overlays to enable auth, while we suggest a pattern to use a kubernetes secret the user needs to manage getting secrets into their system securely on their own

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Dec 10, 2024
Comment on lines +17 to +20
valueFrom:
secretKeyRef:
name: phoenix-secret
key: secret-key
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we default this to just a long string but leave the note? Otherwise it will break... It's what I did with railway... I know it's sorta risky but it at least gets people started (similar to having postgres password)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: 👍 Approved
Development

Successfully merging this pull request may close these issues.

[auth][kubernetes] update kubernetes manifest to deploy phoenix with and without auth
2 participants