Handle JWT generation properly

Would be nice to make the apiRoute thing properly typed so CI would catch this error, but I think that's a challenge for another day.
bluedotimpact · Sep 26, 2024 · ec449dd · ec449dd
1 parent 0a8dbfd
commit ec449dd
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/apps/meet/src/lib/api/db.ts b/apps/meet/src/lib/api/db.ts
@@ -27,7 +27,7 @@ export const cohortTable: Table<Cohort> = {
 };
 
 export interface CohortClass extends Item {
-  'Facilitator': string[],
+  'Facilitators': string[],
   'Participants (Expected)': string[],
   'Attendees': string[],
   'Start date/time': number | null,
@@ -40,15 +40,15 @@ export const cohortClassTable: Table<CohortClass> = {
   baseId: airtableBaseId,
   tableId: 'tblDNME0bA9OoApTk',
   schema: {
-    Facilitator: 'string[]',
+    Facilitators: 'string[]',
     'Participants (Expected)': 'string[]',
     Attendees: 'string[]',
     'Start date/time': 'number | null',
     Cohort: 'string',
     'Zoom account': 'string | null',
   },
   mappings: {
-    Facilitator: 'fldP5BqdFfcn8enfc',
+    Facilitators: 'fldP5BqdFfcn8enfc',
     'Participants (Expected)': 'fldEKYwcacAa6nBEE',
     Attendees: 'fldo0xEi6vJKSJlFN',
     'Start date/time': 'flduTqIxS6OEHNr4H',

diff --git a/apps/meet/src/pages/api/public/meeting-jwt.ts b/apps/meet/src/pages/api/public/meeting-jwt.ts
@@ -45,7 +45,7 @@ export default apiRoute(async (
   const oPayload = {
     sdkKey: env.NEXT_PUBLIC_ZOOM_CLIENT_ID,
     mn: meetingNumber,
-    role: cohortClass.Facilitator === req.body.participantId ? ZOOM_ROLE.HOST : ZOOM_ROLE.PARTICIPANT,
+    role: cohortClass.Facilitators.includes(req.body.participantId) ? ZOOM_ROLE.HOST : ZOOM_ROLE.PARTICIPANT,
     iat: issuedAt,
     exp: expiresAt,
     tokenExp: expiresAt,

diff --git a/apps/meet/src/pages/api/public/meeting-participants.ts b/apps/meet/src/pages/api/public/meeting-participants.ts
@@ -73,7 +73,7 @@ export default apiRoute(async (
     throw new createHttpError.InternalServerError(`Cohort class ${cohortClass.id} missing Zoom account`);
   }
   const zoomAccount = await db.get(zoomAccountTable, cohortClass['Zoom account']);
-  const facilitators = await Promise.all(cohortClass.Facilitator.map((facilitatorId) => db.get(personTable, facilitatorId)));
+  const facilitators = await Promise.all(cohortClass.Facilitators.map((facilitatorId) => db.get(personTable, facilitatorId)));
   const participants = await Promise.all(
     cohortClass['Participants (Expected)']
       .map((participantId) => db.get(personTable, participantId)),