WIP making My Courses work (currently still not authenticated)

Author: Breus

src/main/java/net/hackyourfuture/coursehub/SecurityConfig.java

@@ -37,6 +37,8 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
                         .permitAll()
                         .requestMatchers(HttpMethod.OPTIONS, "/courses")
                         .permitAll()
+                        .requestMatchers("/students/**")
+                        .hasRole("student")
                         .anyRequest()
                         .authenticated())
                 .build();

src/main/java/net/hackyourfuture/coursehub/web/StudentController.java

@@ -1,8 +1,10 @@
 package net.hackyourfuture.coursehub.web;
 
+import jakarta.validation.constraints.Positive;
 import net.hackyourfuture.coursehub.service.CourseService;
 import net.hackyourfuture.coursehub.web.model.CourseDto;
 import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
@@ -19,7 +21,7 @@ public StudentController(CourseService courseService) {
     }
 
     @GetMapping("/{studentId}/courses")
-    public List<CourseDto> getCoursesForStudent(Integer studentId) {
+    public List<CourseDto> getCoursesForStudent(@PathVariable(value = "studentId") @Positive Integer studentId) {
         return courseService.getCoursesForStudent(studentId);
     }
 }

ui/src/pages/MyCourses.tsx

@@ -17,7 +17,7 @@ function MyCourses({user}: { user: User | null }) {
             credentials: "include",
         }).then(res => res.json())
             .then(data => setCourses(data.courses))
-            .catch(err => console.error('Error fetching courses:', err))
+            .catch(err => console.error("Error fetching My Courses:", err))
     }, [])
 
     return (
@@ -40,7 +40,7 @@ function MyCourses({user}: { user: User | null }) {
                     <tbody>
                     {courses.map((course, idx) => (
                         <tr
-                            key={course.courseId}
+                            key={course.id}
                             className={`border-t transition ${idx % 2 === 0 ? 'bg-gray-50' : 'bg-white'} hover:bg-blue-50`}
                         >
                             <td className="px-6 py-4 text-gray-700">{course.name}</td>
@@ -59,5 +59,3 @@ function MyCourses({user}: { user: User | null }) {
 }
 
 export default MyCourses
-
-