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

Statuscake: read basicauth password straight from secret #608

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

matthijswolters-rl
Copy link

This is the first PR of 3 in which I upstream some changes I made to the operator because it was missing functionality that we wanted.

Changes

This PR adds the ability to read basic auth data straight from a secret rather than from an env variable. This is nice because you do not need to restart the operator for it to load in new env variables, it can just load the secret while still running. I have kept the old option because I did not want to introduce any breaking changes. I have also updated the docs and CRDs and such. Let me know if I have missed anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant