Merge pull request #22 from TeaByte/majhcc/issue21

Fix sessionid issue
TeaByte · Jan 16, 2024 · 7ef13c0 · 7ef13c0
2 parents b166b3d + 2772dbd
commit 7ef13c0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/routes/_middleware.ts b/routes/_middleware.ts
@@ -8,7 +8,7 @@ export async function handler(
     const resp = await ctx.next();
     const cookies = getCookies(req.headers);
     const sessionId = cookies["sessionId"] || "";
-    if (!sessionId) {
+    if (sessionId == "") {
         try {
             const student = await createStudent();
             const cookie: Cookie = {

diff --git a/routes/index.tsx b/routes/index.tsx
@@ -21,6 +21,13 @@ export const handler: Handlers<Props> = {
   async GET(_req, ctx) {
     const courses = await getCourses(cache);
     const session = getCookies(_req.headers)["sessionId"];
+    if (!session) {
+      return ctx.render({
+        completed: [],
+        total: getNumberOfCourses(courses.courses),
+        courses: courses.courses,
+      });
+    }
     const completed = (await getStudent(session)).completedCourses;
     const total = getNumberOfCourses(courses.courses);
     return ctx.render({