-
Notifications
You must be signed in to change notification settings - Fork 427
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
Add support for generating htpasswd hash #32
Comments
Can you point me to some documentation? I'm not sure what the hashing algo is that it uses. |
If anyone comes here from google, an easy way to get around it for helm charts and nginx is to use plain text to store the pw:
Not cool to store the PWs like this but it's in a kubernetes secret so i guess it's ok? Suppose for proper auth you wouldn't use basic auth anyway @technosophos The algorithms are a bit of a mess: |
Can you explain where it is stored in plain text? because the secret is base64 encoded |
@kamshak Nginx indeed seems to support plain text passwords in the passwd file. However, when I create a Kubernetes secret that contains Are you sure that your approach should work for the Nginx Ingress controller? |
Hey, I'm not running the app that required this anymore so I'm not sure if
it still works (but I guess it should unless something in nginx changed). I
think the key to making it work was to have that {PLAIN} in there.
Sander Ploegsma <notifications@github.com> schrieb am Di., 10. Dez. 2019,
14:47:
… @kamshak <https://github.com/Kamshak> Nginx indeed seems to support plain
text passwords in the passwd file. However, when I create a Kubernetes
secret that contains foo:bar (base64 encoded) the Nginx Controller
doesn't accept these credentials. If I use htpasswd to encrypt bar it
works.
Are you sure that your approach should work for the Nginx Ingress
controller?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#32?email_source=notifications&email_token=AACSQMAQV5IEH6AXZ5BFVULQX6MWPA5CNFSM4DGQXVPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGPI6HY#issuecomment-564039455>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACSQMHO4MSB6BBN5CWYUR3QX6MWPANCNFSM4DGQXVPA>
.
|
* Add support for generating htpasswd hash Fixes issue #32 * Fix typo
Would it be possible implement third argument for e.g.:
|
A function that can generate the
htpasswd
type hash from username & password would be great. In Kubernetes Helm charts, this would help in configuring basic auth for ingress resources.The text was updated successfully, but these errors were encountered: