-
-
Notifications
You must be signed in to change notification settings - Fork 205
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
Added the possibility to use multiple or other authentication methods #110
base: master
Are you sure you want to change the base?
Conversation
Add auth method to be able tonuse key and password
This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution! Please read this blog post to see the reasons why I mark pull requests as stale. |
This would like it to make it in the code :-) |
This issue is no longer marked for closure. |
This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! Please read this blog post to see the reasons why I mark issues as stale. |
This would like it to make it in the code :-) |
This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! Please read this blog post to see the reasons why I mark issues as stale. |
This would like it to make it in the code :-) |
This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! Please read this blog post to see the reasons why I mark issues as stale. |
@geerlingguy I know you are busy, can you update the stale bot to close stale PR after 3-months or so? |
I set it to auto-close if there's no response for X amount of time after the 'stale' message is posted, and I think X is 90 days. Usually if I don't get to an issue right away (and with hundreds of projects, that's most of the time), it's not affecting me personally and there's a good chance it won't make it in. This particular feature request is something I don't use personally (I don't ever use password auth), but I see there could be some value. I don't think we need the added complexity of automatic identification of whether password auth should be set to 'yes' or 'no' though. It would be a simpler PR to review/maintain if it just added the new var and kept the existing one set to 'no'. |
No description provided.