Merge pull request #3 from simayosi/fix/removal-unauthenticated-bind

k2u2 · web-flow · commit b3751465ee41 · 2025-02-12T14:11:21.000+09:00
Removed the LDAP UnauthenticatedBind() call.
diff --git a/src/ngx_auth/ldap_auth/bind_auth.go b/src/ngx_auth/ldap_auth/bind_auth.go
@@ -135,7 +135,6 @@ func (lba *LdapAuth) Authenticate(user, pass string) (bool, bool, error) {
 	if lba.conn.Bind(bind_dn, pass) != nil {
 		return false, false, nil
 	}
-	defer lba.conn.UnauthenticatedBind(bind_dn)
 
 	if lba.cfg.UniqueFilter != "" {
 		res, e := lba.conn.Search(lba.new_search_param(lba.cfg.UniqueFilter, user))

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,6 @@ func (lba *LdapAuth) Authenticate(user, pass string) (bool, bool, error) {`
`135`	`135`	`if lba.conn.Bind(bind_dn, pass) != nil {`
`136`	`136`	`return false, false, nil`
`137`	`137`	`}`
`138`		`- defer lba.conn.UnauthenticatedBind(bind_dn)`
`139`	`138`
`140`	`139`	`if lba.cfg.UniqueFilter != "" {`
`141`	`140`	`res, e := lba.conn.Search(lba.new_search_param(lba.cfg.UniqueFilter, user))`