goauthentik / authentik

The authentication glue you need.

Home Page:https://goauthentik.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Make semicolon separation optional

Lokkenum opened this issue · comments

Hello there!

According to official documentation:
https://docs.goauthentik.io/docs/providers/ldap/#bind-modes

For code-based authenticators, the code must be given as part of the bind password, separated by a semicolon. For example for the password example-password and the code 123456, the input must be example-password;123456.

What is the reason and purpose for semicolon separation?
I'm getting a ton of negative feedback from users especially from notebooks and mobile devices who are forgetting to add semicolon.

How to get rid of this requirement or make this optional?
Should be a quick fix/feature.

Thank you!

Before we introduced the option to make support for MFA in LDAP a toggle that can be enabled, having the semicolon required was part of the detection for MFA codes

Before we introduced the option to make support for MFA in LDAP a toggle that can be enabled, having the semicolon required was part of the detection for MFA codes

Strange requirement.
What will happen if user password contains semicolon like in this example;password;123456 ?
Why just not to parse last 6-8 digit's from password string?