Skip to content
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

Ldap users wrong email validation #13462

Open
mehdi-ra opened this issue Feb 24, 2025 · 1 comment
Open

Ldap users wrong email validation #13462

mehdi-ra opened this issue Feb 24, 2025 · 1 comment
Labels
in linear Issue or PR has been created in Linear for internal review In Review We are reviewing the issue

Comments

@mehdi-ra
Copy link

Bug Description

I migrated directly from n8n 1.66 to latest version (1.79.3) and we encountered a problem with LDAP users.
It seems validation for email is still applied to ldap usernames too and it keeps us from logging in with invalid email error witch comes from zod validator defined at `packages/@n8n/api-types/src/dto/auth/login-request.dto.ts

To Reproduce

  • Enable ldap on older versions like 1.66
  • Migrate to version 1.79.3
  • Try to login using ldap user username.

Expected behavior

Ldap users should be able to use their domain username to log in and validation should be passed for email.

Operating System

N8N docker - Kubernetes

n8n Version

1.79.3

Node.js Version

20.0.0

Database

PostgreSQL

Execution mode

main (default)

@Joffcom
Copy link
Member

Joffcom commented Feb 24, 2025

Hey @mehdi-ra,

We have created an internal ticket to look into this which we will be tracking as "GHC-940"

@Joffcom Joffcom added in linear Issue or PR has been created in Linear for internal review In Review We are reviewing the issue labels Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in linear Issue or PR has been created in Linear for internal review In Review We are reviewing the issue
Projects
None yet
Development

No branches or pull requests

2 participants