case sensitivity on user emails (#93)

* removed case sensitivity on user emails while listing bindings * chore(workflow): add for access-management * resolve build conflicts * fix workflow * added lowercasing to emails * reverted * removed redundant lowercase * reverted changes * added branch to build * arbitrary change * removed branch name from gthub workflow * removed arbitrary comment Co-authored-by: Mathis Marcotte <mathis.marcotte@statcan.gc.ca> Co-authored-by: Jose Matsuda <jose.matsuda@statcan.gc.ca> Co-authored-by: Jose-Matsuda <jose.matsuda@canada.ca>
StatCan · Sep 20, 2022 · f02e4d1 · f02e4d1
1 parent 689f07c
commit f02e4d1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/components/centraldashboard/app/api_workgroup.ts b/components/centraldashboard/app/api_workgroup.ts
@@ -210,7 +210,7 @@ export class WorkgroupApi {
         let errIndex = 0;
         try {
             const binding = mapSimpleBindingToWorkgroupBinding({
-                user: contributor,
+                user: contributor.toLowerCase(),
                 namespace,
                 role: 'contributor',
             });
@@ -277,6 +277,7 @@ export class WorkgroupApi {
             const profile = req.body as CreateProfileRequest;
             try {
                 const namespace = profile.namespace || req.user.username;
+                const owerName = profile.user || req.user.email;
                 // Use the request body if provided, fallback to auth headers
                 await this.profilesService.createProfile({
                     metadata: {
@@ -285,7 +286,7 @@ export class WorkgroupApi {
                     spec: {
                         owner: {
                             kind: 'User',
-                            name: profile.user || req.user.email,
+                            name: owerName.toLowerCase(),
                         }
                     },
                 });

diff --git a/components/centraldashboard/app/attach_user_middleware.ts b/components/centraldashboard/app/attach_user_middleware.ts
@@ -13,6 +13,7 @@ export function attachUser(
       email = req.header(userIdHeader).slice(userIdPrefix.length);
       auth = {[userIdHeader]: req.header(userIdHeader)};
     }
+    email = email.toLowerCase();
     req.user = {
       email,
       username: email.split('@')[0],