Fix authentication bugs

backend/project/endpoints/authentication/auth.py

@@ -45,6 +45,12 @@ def microsoft_authentication():
         res = requests.post(f"https://login.microsoftonline.com/{TENANT_ID}/oauth2/v2.0/token",
                             data=data,
                             timeout=5)
+        if res.status_code != 200:
+            abort(make_response((
+                {"message":
+                    "An error occured while trying to authenticate your access token"},
+                    500)))
+        # hier wel nog if om error zelf op te vangen
         token = res.json()["access_token"]
         profile_res = requests.get("https://graph.microsoft.com/v1.0/me",
                                     headers={"Authorization":f"Bearer {token}"},

backend/project/init_auth.py

@@ -19,7 +19,7 @@ def expired_token_callback(jwt_header, jwt_payload):
             401)
 
     @jwt.invalid_token_loader
-    def invalid_token_callback(jwt_header, jwt_payload):
+    def invalid_token_callback(jwt_header):
         return (
             {"message":("The server cannot recognize this access token cookie, "
                         "please log in again if you think this is an error")},

frontend/src/components/Courses/CourseDetailTeacher.tsx

@@ -6,6 +6,7 @@ import { Link, useNavigate, NavigateFunction, useLoaderData } from "react-router
 import { Title } from "../Header/Title";
 import ClearIcon from '@mui/icons-material/Clear';
 import { timeDifference } from "../../utils/date-utils";
+import { authenticatedFetch } from "../../utils/authenticated-fetch";
 
 interface UserUid{
     uid: string
@@ -18,12 +19,8 @@ interface UserUid{
  * @param uid - The UID of the admin.
  */
 function handleDeleteAdmin(navigate: NavigateFunction, courseId: string, uid: string): void {
-  fetch(`${apiHost}/courses/${courseId}/admins`, {
+  authenticatedFetch(`${apiHost}/courses/${courseId}/admins`, {
     method: 'DELETE',
-    credentials: 'include',
-    headers: {
-      "Content-Type": "application/json"
-    },
     body: JSON.stringify({
       "admin_uid": uid
     })
@@ -40,11 +37,10 @@ function handleDeleteAdmin(navigate: NavigateFunction, courseId: string, uid: st
  * @param uid - The UID of the admin.
  */
 function handleDeleteStudent(navigate: NavigateFunction, courseId: string, uids: string[]): void {
-  fetch(`${apiHost}/courses/${courseId}/students`, {
+  authenticatedFetch(`${apiHost}/courses/${courseId}/students`, {
     method: 'DELETE',
-    credentials: 'include',
     headers: {
-      "Content-Type": "application/json"
+      "Content-Type": "application/json",
     },
     body: JSON.stringify({
       "students": uids
@@ -61,9 +57,8 @@ function handleDeleteStudent(navigate: NavigateFunction, courseId: string, uids:
  * @param courseId - The ID of the course.
  */
 function handleDeleteCourse(navigate: NavigateFunction, courseId: string): void {
-  fetch(`${apiHost}/courses/${courseId}`, {
+  authenticatedFetch(`${apiHost}/courses/${courseId}`, {
     method: 'DELETE',
-    credentials: 'include',
   }).then((response) => {
     if(response.ok){
       navigate(-1);
@@ -289,9 +284,8 @@ function JoinCodeMenu({courseId,open,handleClose, anchorEl}: {courseId:string, o
   };
 
   const getCodes = useCallback(() => {
-    fetch(`${apiHost}/courses/${courseId}/join_codes`, {
+    authenticatedFetch(`${apiHost}/courses/${courseId}/join_codes`, {
       method: 'GET',
-      credentials: 'include',
     })
       .then(response => response.json())
       .then(data => {
@@ -317,24 +311,22 @@ function JoinCodeMenu({courseId,open,handleClose, anchorEl}: {courseId:string, o
       bodyContent.expiry_time = expiry_time.toISOString();
     }
 
-    fetch(`${apiHost}/courses/${courseId}/join_codes`, {
+    authenticatedFetch(`${apiHost}/courses/${courseId}/join_codes`, {
       method: 'POST',
-      credentials: 'include',
       headers: {
-        'Content-Type': 'application/json'
+        'Content-Type': 'application/json',
       },
       body: JSON.stringify(bodyContent)
     })
       .then(() => getCodes())
   }
 
   const handleDeleteCode = (joinCode: string) => {
-    fetch(`${apiHost}/courses/${courseId}/join_codes/${joinCode}`,
+    authenticatedFetch(`${apiHost}/courses/${courseId}/join_codes/${joinCode}`,
       {
         method: 'DELETE',
-        credentials: 'include',
         headers: {
-          "Content-Type": "application/json"
+          "Content-Type": "application/json",
         },
         body: JSON.stringify({
           "join_code": joinCode