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

Option to handle dex-core client_secret via k8s secrets + alpine update #184

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

Conversation

VF-mbrauer
Copy link

Added functionality to overcome a security query where client_secretis in plain text inside the config.yaml, which is stored in a config map. So now it needs to be stored as a Base64 Encoded String and then the k8s-authenticator will decrypt internally and send the secret then towards the dexidp.

Also, the Alpine version has been increased as well as the documentation is updated for the client_secret change.

@VF-mbrauer VF-mbrauer changed the title Added function to encode client_secret in base64 + alpine update Option to handle dex-core client_secret via k8s secrets + alpine update Jan 12, 2022
@VF-mbrauer
Copy link
Author

@nabadger, I have changed the concept as of our last conversation in #179
So please review again and let me know if something is unclear.
Thanks a lot.

@VF-mbrauer
Copy link
Author

@nabadger any news? Did you check already?

@VF-mbrauer
Copy link
Author

@nabadger Is that repo still under maintainer mode? No answer since month already. So is here any chance to get an update? Otherwise we need to think of a product change as we need some support with the current one. Thanks a lot.

@VF-mbrauer
Copy link
Author

  • @nabadger PR is open since almost 4 month. Any chance to start working my PR?

@VF-mbrauer
Copy link
Author

@nickmintel Can you please have a look? Thanks.

@nabadger
Copy link
Contributor

nabadger commented Mar 9, 2022

@VF-mbrauer apologies - I've not been able to commit any time for this project recently (I'm not sure these circumstances will change either for a while).

After talking to internal team here, I've decided to raise a ticket to see if we can find other owners/maintainers of this project:

#194

@VF-mbrauer
Copy link
Author

Hi @nabadger, fair point and I absolutely understand. So, no worries. Would be great if we could find other folks who take over as this product is great enrichment for the DEX-Core.

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.

None yet

2 participants