Can't login as user@company.com with ldap authentication enabled #2599

New Issue

GiteaMirror · 2025-11-02T04:41:39-06:00

GiteaMirror commented

2025-11-02 04:41:39 -06:00

Originally created by @ghost on GitHub (Dec 1, 2018).

Gitea version (or commit ref): v1.6.0
Git version: 2.17.1
Operating system: Ubuntu 18.04.1
Database (use [x]):
- PostgreSQL
- MySQL
- MSSQL
- SQLite
Can you reproduce the bug at https://try.gitea.io:
- Yes (provide example URL)
- No
- Not relevant
Log gist:

xxxx/xx/xx xx:xx:xx [W] Failed to login 'user@company.com' via 'local_ldap': Invalid pattern for attribute 'username' [user@company.com]: must be valid alpha or numeric or dash(-_) or dot characters
xxxx/xx/xx xx:xx:xx [I] Failed authentication attempt for user@company.com from xxx.xxx.xxx.xxx

Description

Additional information:

OpenLDAP: 2.4.45
Authentication type: LDAP (simple auth)
User DN: uid=%s,ou=my_ou,dc=example,dc=com
User Filter: (&(objectClass=inetOrgPerson)(uid=%s))

Obviously the "@" character is not in a class of allowed characters for "username". Will it be possible to add the "@" in the next version?

Originally created by @ghost on GitHub (Dec 1, 2018).  - Gitea version (or commit ref): v1.6.0 - Git version: 2.17.1 - Operating system: Ubuntu 18.04.1 - Database (use `[x]`): - [ ] PostgreSQL - [ ] MySQL - [ ] MSSQL - [x] SQLite - Can you reproduce the bug at https://try.gitea.io: - [ ] Yes (provide example URL) - [ ] No - [x] Not relevant - Log gist: ``` xxxx/xx/xx xx:xx:xx [W] Failed to login 'user@company.com' via 'local_ldap': Invalid pattern for attribute 'username' [user@company.com]: must be valid alpha or numeric or dash(-_) or dot characters xxxx/xx/xx xx:xx:xx [I] Failed authentication attempt for user@company.com from xxx.xxx.xxx.xxx ``` ## Description Additional information: * OpenLDAP: 2.4.45 * Authentication type: LDAP (simple auth) * User DN: uid=%s,ou=my_ou,dc=example,dc=com * User Filter: (&(objectClass=inetOrgPerson)(uid=%s)) Obviously the "@" character is not in a class of allowed characters for "username". Will it be possible to add the "@" in the next version?

GiteaMirror added the type/question label 2025-11-02 04:41:39 -06:00

GiteaMirror closed this issue

2025-11-02 04:41:40 -06:00

GiteaMirror commented

2025-11-02 04:41:40 -06:00

@ghost commented on GitHub (Dec 1, 2018):

Sorry, I put the issue in the comment field instead of the description field.

@ghost commented on GitHub (Dec 1, 2018): Sorry, I put the issue in the comment field instead of the description field.

GiteaMirror commented

2025-11-02 04:41:40 -06:00

@lafriks commented on GitHub (Dec 2, 2018):

There was already request to add this but @ can not be added as allowed character as could lead to side effects in other places later on. It would be ok tho to have it replaced or domain part removed for username with additional option

@lafriks commented on GitHub (Dec 2, 2018): There was already request to add this but `@` can not be added as allowed character as could lead to side effects in other places later on. It would be ok tho to have it replaced or domain part removed for username with additional option

GiteaMirror commented

2025-11-02 04:41:41 -06:00

@ghost commented on GitHub (Dec 3, 2018):

I have seen that you can also authenticate via SMTP. For now this is a workaround. Thanks for the quick answer.

@ghost commented on GitHub (Dec 3, 2018): I have seen that you can also authenticate via SMTP. For now this is a workaround. Thanks for the quick answer.

GiteaMirror commented

2025-11-02 04:41:41 -06:00

@techknowlogick commented on GitHub (Dec 4, 2018):

Closing this issue as a workaround is present. If you have any follow up questions please feel free to open a thread on the forum

@techknowlogick commented on GitHub (Dec 4, 2018): Closing this issue as a workaround is present. If you have any follow up questions please feel free to open a thread on the [forum](https://discourse.gitea.io/)

GiteaMirror referenced this issue

2025-11-02 12:11:59 -06:00

[PR #2599] [MERGED] Fix translation upload to crowdin #16496

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/gitea#2599