chore: Tidy up

theopensystemslab · Sep 3, 2023 · 58ba9e9 · 58ba9e9
1 parent 18b72a3
commit 58ba9e9
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/api.planx.uk/modules/auth/middleware.ts b/api.planx.uk/modules/auth/middleware.ts
@@ -143,7 +143,9 @@ export const useRoleAuth: UseRoleAuth =
 
       if (!isAuthorised) {
         console.error(
-          `Authentication error: User ${userId} does have have any of the roles ${authRoles} which are required to access ${req.path}`,
+          `Authentication error: User ${userId} does have have any of the roles [${authRoles.join(
+            ", ",
+          )}] which are required to access ${req.path}`,
         );
         return next({
           status: 403,

diff --git a/api.planx.uk/modules/auth/service.ts b/api.planx.uk/modules/auth/service.ts
@@ -19,8 +19,8 @@ export const buildJWT = async (email: string | undefined) => {
   const { id } = users[0];
 
   const hasura = {
-    "x-hasura-allowed-roles": ["admin"],
-    "x-hasura-default-role": "admin",
+    "x-hasura-allowed-roles": ["platformAdmin", "public"],
+    "x-hasura-default-role": "platformAdmin",
     "x-hasura-user-id": id.toString(),
   };
 

diff --git a/api.planx.uk/modules/auth/strategy/google.ts b/api.planx.uk/modules/auth/strategy/google.ts
@@ -14,7 +14,7 @@ export const googleStrategy = new GoogleStrategy(
     if (!jwt) {
       return done({
         status: 404,
-        message: `User (${email}) not found.Do you need to log in to a different Google Account?`,
+        message: `User (${email}) not found. Do you need to log in to a different Google Account?`,
       } as any);
     }