Can not update LDAP with simple auth after creation #3160

Closed
opened 2025-11-02 05:02:22 -06:00 by GiteaMirror · 2 comments
Owner

Originally created by @iddm on GitHub (Apr 9, 2019).

After creating an LDAP authentication resource with simple authentication method I found several bugs:

  1. During the creation, when I switch to LDAP (simple auth) there is still Search Base field there without red star above it. According to the documentation, this field should be hidden.
  2. Even though creation goes okay, whenever I try to edit it, this Search Base field is shown and prevents me from submitting changes saying "This field must be filled".
  • Gitea version d7542bf:
  • Git version: not relevant
  • Operating system: not relevant
  • Database (use [x]):
    • PostgreSQL
    • MySQL (MariaDB)
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
Originally created by @iddm on GitHub (Apr 9, 2019). After creating an LDAP authentication resource with simple authentication method I found several bugs: 1. During the creation, when I switch to `LDAP (simple auth)` there is still `Search Base` field there without red star above it. According to the documentation, this field should be hidden. 2. Even though creation goes okay, whenever I try to edit it, this `Search Base` field is shown and prevents me from submitting changes saying "This field must be filled". - Gitea version d7542bf: - Git version: not relevant - Operating system: not relevant - Database (use `[x]`): - [ ] PostgreSQL - [x] MySQL (MariaDB) - [ ] MSSQL - [ ] SQLite - Can you reproduce the bug at https://try.gitea.io: - [ ] Yes (provide example URL) - [ ] No - [x] Not relevant
GiteaMirror added the topic/uitype/bug labels 2025-11-02 05:02:22 -06:00
Author
Owner

@iddm commented on GitHub (Apr 19, 2019):

This is really an annoying bug, guys. Right now many people who use the gitea and ldap + basic authentication are suffering from this user interface bug. This should be really simple to fix.

@iddm commented on GitHub (Apr 19, 2019): This is really an annoying bug, guys. Right now many people who use the gitea and ldap + basic authentication are suffering from this user interface bug. This should be really simple to fix.
Author
Owner

@ngourdon commented on GitHub (Apr 27, 2019):

It seems the Search Base field is used with LDAP (simple auth) but it shouldn't be required.
https://github.com/go-gitea/gitea/blob/master/modules/auth/ldap/ldap.go#L206

@ngourdon commented on GitHub (Apr 27, 2019): It seems the `Search Base` field is used with `LDAP (simple auth)` but it shouldn't be required. https://github.com/go-gitea/gitea/blob/master/modules/auth/ldap/ldap.go#L206
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#3160