rest: follow up from previous commit, make sure the username is corre…

…ctly formatted for external accounts
cernbox · Sep 17, 2024 · a46908a · a46908a
1 parent aa7e67b
commit a46908a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/user/rest.go b/user/rest.go
@@ -224,12 +224,12 @@ func (m *manager) parseAndCacheUser(ctx context.Context, i *Identity) (*userpb.U
 			Idp:      m.conf.IDProvider,
 			Type:     i.UserType(),
 		},
-		Username:    i.Upn,
 		Mail:        i.PrimaryAccountEmail,
 		DisplayName: i.DisplayName,
 		UidNumber:   int64(i.UID),
 		GidNumber:   int64(i.GID),
 	}
+	u.Username = utils.FormatUserID(u.Id)
 
 	if err := m.cacheUserDetails(u); err != nil {
 		log.Error().Err(err).Msg("rest: error caching user details")