-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
ECDSA keys don't work with LDAP sync #5092
Labels
Milestone
Comments
Hi Guys, I could create a PR for this as I have tested it and works fine with parsing in a normal way instead of checking for a limited Just let me know. Kind regards, |
@xor-gate PRs are always welcome 😄 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
[x]
):Description
LDAP sync doesn't work with ECDSA keys because the algo string check is hardcoded instead of using ssh.ParseAuthorizedKey.
Solution
https://github.com/go-gitea/gitea/blob/master/models/user.go#L1458
Should become
The text was updated successfully, but these errors were encountered: