feat: add optional function for LDAPBasicAuthIdentityProvider to prov…

…ide different userid in case it differs from dn

plugins/ldap/src/LDAPBasicAuthIdentityProvider.ts

@@ -24,12 +24,16 @@ export class LDAPBasicAuthIdentityProvider implements IdentityProvider {
 
   /**
    *
-   * @param DNforUsername should be a function that formats a username into a DN for use in an LDAP bind.
+   * @param DNForUsername should be a function that formats a username into a DN for use in an LDAP bind.
    * e.g. (name) => `uid=${name},ou=Users`;
+   * @param SubForUsername optional function to map username to a different sub (id). Use this if your user lookup
+   * uses a different attribute.
    */
   constructor(
     private ldapSettings: LDAPClientSettings,
-    private DNforUsername: (username: string) => string
+    private DNForUsername: (username: string) => string,
+    private SubForUsername: (username: string) => string = (username) =>
+      username
   ) {}
 
   async params(): Promise<IdentityProviderParams> {
@@ -51,7 +55,7 @@ export class LDAPBasicAuthIdentityProvider implements IdentityProvider {
 
     const ldap = await initLdapClient(this.ldapSettings);
 
-    const dn = this.DNforUsername(name);
+    const dn = this.DNForUsername(name);
 
     try {
       await ldap.bind(dn, pass);
@@ -69,7 +73,7 @@ export class LDAPBasicAuthIdentityProvider implements IdentityProvider {
     return {
       status: "ok",
       identity: {
-        sub: name,
+        sub: this.SubForUsername(name),
       },
     };
   }