Skip to content

LDAP Authentication#346

Closed
ghost wants to merge 79 commits intomasterfrom
unknown repository
Closed

LDAP Authentication#346
ghost wants to merge 79 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 6, 2023

Limitations

  • Currently the bind user's password is stored unencrypted, this is a limitation of LDAP.

Fixes: #44.

@ghost ghost force-pushed the ldap_auth branch from d139471 to 80f5fac Compare August 6, 2023 10:13
@ghost ghost changed the title [WIP] LDAP Authentication LDAP Authentication Aug 6, 2023
@sentriz sentriz modified the milestone: v0.16.0 Aug 31, 2023
@sentriz
Copy link
Copy Markdown
Owner

sentriz commented Sep 7, 2023

thanks for the effort here 👍

it also may take some time for me to review since i don't know anything about LDAP. and i don't think i should blindy accept something i don't know about (no offense 😁) i will do some reading or get a second pair of eyeballs on it

there seems to be some lint errors btw

@ghost
Copy link
Copy Markdown
Author

ghost commented Apr 7, 2024

You can implement a server-side auth caching method.

@nitnelave, this has been done in fccf016.

Copy link
Copy Markdown
Owner

@sentriz sentriz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi there seems like there some lint errors

Comment thread cmd/gonic/gonic.go Outdated
@sentriz sentriz force-pushed the master branch 3 times, most recently from c5b3a1c to 86fd590 Compare May 15, 2024 15:07
@yaemiku yaemiku mentioned this pull request Sep 21, 2025
@sentriz sentriz force-pushed the master branch 5 times, most recently from 7decea6 to 971d22d Compare November 11, 2025 07:01
@ghost ghost closed this by deleting the head repository Mar 8, 2026
lurenyang418 pushed a commit to lurenyang418/gonic that referenced this pull request Mar 23, 2026
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

external user authentification

3 participants