Ldap fix (#412)

* fixed missing username for ldap

* play with lint

CONTRIBUTORS.md

@@ -7,4 +7,5 @@
 - Hugo Gonzalez Labrador <github@hugo.labkode.com>
 - Ilja Neumann <ineumann@owncloud.com>
 - Jörn Friedrich Dreyer <jfd@butonic.de>
+- Michael D'Silva <michael.dsilva@aarnet.edu.au>
 - Mohitty <mohitt@iitk.ac.in>

pkg/auth/manager/ldap/ldap.go

@@ -55,7 +55,6 @@ type config struct {
 type attributes struct {
 	DN          string `mapstructure:"dn"`
 	UID         string `mapstructure:"uid"`
-	UserName    string `mapstructure:"userName"`
 	Mail        string `mapstructure:"mail"`
 	DisplayName string `mapstructure:"displayName"`
 }
@@ -141,12 +140,13 @@ func (am *mgr) Authenticate(ctx context.Context, clientID, clientSecret string)
 			OpaqueId: sr.Entries[0].GetAttributeValue(am.c.Schema.UID),
 		},
 		// TODO add more claims from the StandardClaims, eg EmailVerified
-		Username: sr.Entries[0].GetAttributeValue(am.c.Schema.UserName),
+		Username: sr.Entries[0].GetAttributeValue(am.c.Schema.UID),
 		// TODO groups
 		Groups:      []string{},
 		Mail:        sr.Entries[0].GetAttributeValue(am.c.Schema.Mail),
 		DisplayName: sr.Entries[0].GetAttributeValue(am.c.Schema.DisplayName),
 	}
+
 	return u, nil
 
 }

pkg/user/manager/ldap/ldap.go

@@ -139,7 +139,7 @@ func (m *manager) GetUser(ctx context.Context, uid *userpb.UserId) (*userpb.User
 
 	log.Debug().Interface("entries", sr.Entries).Msg("entries")
 
-	return &userpb.User{
+	u := &userpb.User{
 		Id: &userpb.UserId{
 			Idp:      m.idp,
 			OpaqueId: uid.OpaqueId,
@@ -148,7 +148,9 @@ func (m *manager) GetUser(ctx context.Context, uid *userpb.UserId) (*userpb.User
 		Groups:      []string{},
 		Mail:        sr.Entries[0].GetAttributeValue(m.schema.Mail),
 		DisplayName: sr.Entries[0].GetAttributeValue(m.schema.DisplayName),
-	}, nil
+	}
+
+	return u, nil
 }
 
 func (m *manager) FindUsers(ctx context.Context, query string) ([]*userpb.User, error) {