changed some stuff lol

compose.yaml

@@ -8,6 +8,6 @@ services:
       - 'POSTGRES_USER=test'
     volumes:
       - ./init.sql:/docker-entrypoint-initdb.d/init.sql
-      - .db:/var/lib/postgresql/data
+      - ./.db:/var/lib/postgresql/data
     ports:
       - '5432:5432'

init.sql

@@ -1,5 +1,6 @@
 -- This script was generated by the ERD tool in pgAdmin 4.
 -- Please log an issue at https://github.com/pgadmin-org/pgadmin4/issues/new/choose if you find any bugs, including reproduction steps.
+CREATE EXTENSION IF NOT EXISTS "uuid-ossp";
 BEGIN;
 
 
@@ -12,16 +13,19 @@ CREATE TABLE IF NOT EXISTS public.alt_subject
     CONSTRAINT alt_subject_name_key UNIQUE (name)
 );
 
+
+
 CREATE TABLE IF NOT EXISTS public.entries
 (
-    id text COLLATE pg_catalog."default" NOT NULL,
+    id uuid NOT NULL,
     subject_id uuid NOT NULL,
-    title text COLLATE pg_catalog."default",
-    info_entry text COLLATE pg_catalog."default",
-    paragraph text COLLATE pg_catalog."default",
+    title text,
+    info_entry text,
+    paragraph text,
     time_published timestamp without time zone NOT NULL,
-    filepath text COLLATE pg_catalog."default",
-    group_name text COLLATE pg_catalog."default",
+    filepath text,
+    group_name text,
+
     CONSTRAINT entries_pkey PRIMARY KEY (id)
 );
 

src/main/java/dev/ksan/etfoglasiserver/EtfoglasiServerApplication.java

@@ -3,6 +3,7 @@ package dev.ksan.etfoglasiserver;
 import dev.ksan.etfoglasiserver.service.Scraper;
 import java.util.Scanner;
 
+import dev.ksan.etfoglasiserver.service.SubjectService;
 import dev.ksan.etfoglasiserver.util.SubjectLoader;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.CommandLineRunner;
@@ -21,16 +22,18 @@ public class EtfoglasiServerApplication {
 
     Scraper scraper = context.getBean(Scraper.class);
 
-/*
+    SubjectService subjectService = context.getBean(SubjectService.class);
+
+      if (subjectService.count() <= 1) {
           SubjectLoader subjectLoader = context.getBean(SubjectLoader.class);
 
           subjectLoader.loadSubjects();
 
           System.out.println("Loading...");
           subjectLoader.loadAlts();
-
-
- */
+      } else {
+          System.out.println("Subjects already loaded, skipping...");
+      }
 
     Thread webClientThread = new Thread(scraper, "WebClientThread");
     Scanner scanner = new Scanner(System.in);

src/main/java/dev/ksan/etfoglasiserver/controller/SubjectController.java

@@ -17,7 +17,7 @@ import org.springframework.web.bind.annotation.RestController;
 public class SubjectController {
   @Autowired SubjectService service;
 
-  @GetMapping("/subject")
+  @GetMapping("/subjects")
   public List<Subject> getSubjects() {
     return service.getSubjects();
   }

src/main/java/dev/ksan/etfoglasiserver/controller/UserController.java

@@ -18,12 +18,12 @@ public class UserController {
 
   @Autowired UserService service;
 
-  @GetMapping("/users")
-  public ResponseEntity<List<UserDTO>> getUsers() {
-
-    return new ResponseEntity(service.getUsers(), HttpStatus.OK);
-  }
-
+//  @GetMapping("/users")
+//  public ResponseEntity<List<UserDTO>> getUsers() {
+//
+//    return new ResponseEntity(service.getUsers(), HttpStatus.OK);
+//  }
+//
   @GetMapping("users/{userId}")
   public ResponseEntity<UserDTO> getUserById(@PathVariable UUID userId) {
 
@@ -38,8 +38,6 @@ public class UserController {
   @PutMapping("/users")
   public ResponseEntity<UserDTO> updateUser(@RequestBody UserCreationDTO user) {
 
-    NotificationMethod method = NotificationMethod.valueOf(user.getNotification());
-    user.setNotificationMethod(method);
     UserDTO userUpdated = service.updateUser(user);
     if (userUpdated != null) {
       return new ResponseEntity<>(userUpdated, HttpStatus.OK);
@@ -47,6 +45,27 @@ public class UserController {
     return new ResponseEntity<>(HttpStatus.BAD_REQUEST);
   }
 
+    @PutMapping("/users/{userId}/subjects/{subjectId}")
+    public ResponseEntity<UserDTO> addSubject(
+            @PathVariable UUID userId,
+            @PathVariable UUID subjectId
+    ) {
+
+        UserDTO updatedUser = service.addSubject(userId, subjectId);
+
+        return ResponseEntity.ok(updatedUser);
+    }
+
+    @DeleteMapping("/users/{userId}/subjects/{subjectId}")
+    public ResponseEntity<UserDTO> removeSubject(
+            @PathVariable UUID userId,
+            @PathVariable UUID subjectId
+    ) {
+
+        UserDTO updatedUser = service.removeSubject(userId, subjectId);
+
+        return ResponseEntity.ok(updatedUser);
+    }
 
   @PostMapping("/login")
   public ResponseEntity<JwtResponseDTO> login(@RequestBody UserLoginDTO user) {
@@ -63,11 +82,11 @@ public class UserController {
     if (newUser != null) return new ResponseEntity<>(newUser, HttpStatus.CREATED);
     return new ResponseEntity<>(HttpStatus.BAD_REQUEST);
   }
-
-  @DeleteMapping("/users/{userId}")
-  public void deleteUser(@PathVariable UUID userId) {
-    service.deleteUser(userId);
-  }
+//
+//  @DeleteMapping("/users/{userId}")
+//  public void deleteUser(@PathVariable UUID userId) {
+//    service.deleteUser(userId);
+//  }
 
   /*
   @PostMapping("/users")

src/main/java/dev/ksan/etfoglasiserver/dto/JwtResponseDTO.java

@@ -4,14 +4,20 @@ public class JwtResponseDTO {
 
   private String token;
   private String email;
+  private String userId;
   private String message;
 
-  public JwtResponseDTO(String token, String email, String message) {
+  public JwtResponseDTO(String token, String email, String userId, String message) {
     this.token = token;
     this.email = email;
+    this.userId = userId;
     this.message = message;
   }
 
+    public String getUserId() {
+        return userId;
+    }
+
     public String getToken() {
     return token;
   }

src/main/java/dev/ksan/etfoglasiserver/dto/UserCreationDTO.java

@@ -11,7 +11,6 @@ public class UserCreationDTO {
   private String password;
   private Set<Subject> subjectSet;
   private String notification;
-  private NotificationMethod notificationMethod;
 
   public String getEmail() {
     return email;
@@ -54,11 +53,4 @@ public class UserCreationDTO {
     }
 
 
-  public void setNotificationMethod(NotificationMethod method) {
-
-    this.notificationMethod = method;
-  }
-  public NotificationMethod getNotificationMethod() {
-    return notificationMethod;
-  }
 }

src/main/java/dev/ksan/etfoglasiserver/dto/UserDTO.java

@@ -9,7 +9,6 @@ public class UserDTO {
   private UUID id;
   private String email;
   private Set<Subject> subjectSet;
-  private NotificationMethod notificationMethod;
 
   public UserDTO() {}
 
@@ -44,11 +43,5 @@ public class UserDTO {
     this.subjectSet = subjectSet;
   }
 
-  public NotificationMethod getNotificationMethod() {
-    return notificationMethod;
-  }
 
-  public void setNotificationMethod(NotificationMethod notificationMethod) {
-    this.notificationMethod = notificationMethod;
-  }
 }

src/main/java/dev/ksan/etfoglasiserver/model/User.java

@@ -31,7 +31,7 @@ public class User {
   @Column(name = "notification_type", nullable = false)
   private NotificationMethod notificationType = NotificationMethod.NO_NOTIFICATION;
 
-  @ManyToMany(fetch = FetchType.LAZY, cascade = CascadeType.ALL)
+  @ManyToMany(fetch = FetchType.LAZY )
   @JoinTable(name = "user-subject", joinColumns = @JoinColumn(name = "user_id"),
   inverseJoinColumns = @JoinColumn(name = "subject_id"))
   private Set<Subject> subjectSet;

src/main/java/dev/ksan/etfoglasiserver/service/JWTService.java

@@ -18,6 +18,7 @@ import org.springframework.stereotype.Service;
 @Service
 public class JWTService {
 
+    //TODO add persistent token env variable
   private String secretKey = "";
 
   private JWTService() {

src/main/java/dev/ksan/etfoglasiserver/service/Scraper.java

@@ -20,6 +20,11 @@ import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
 import org.springframework.stereotype.Service;
 
+
+// This exists because i made it a while ago and I just kept it just because
+// should probably move to just hitting the endpoints directly instead of digging through the html manually
+
+
 @Component
 public class Scraper implements Runnable {
 

src/main/java/dev/ksan/etfoglasiserver/service/SubjectService.java

@@ -90,4 +90,9 @@ public class SubjectService {
   public void sendPushNotification(Entry entry,User user) {
     System.out.println("Sending push notification");
   }
+
+    public long count() {
+
+      return subjectRepo.count();
+    }
 }

src/main/java/dev/ksan/etfoglasiserver/service/UserService.java

@@ -4,8 +4,9 @@ import dev.ksan.etfoglasiserver.dto.JwtResponseDTO;
 import dev.ksan.etfoglasiserver.dto.UserCreationDTO;
 import dev.ksan.etfoglasiserver.dto.UserDTO;
 import dev.ksan.etfoglasiserver.dto.UserLoginDTO;
+import dev.ksan.etfoglasiserver.model.Subject;
 import dev.ksan.etfoglasiserver.model.User;
-import dev.ksan.etfoglasiserver.model.UserPrincipal;
+import dev.ksan.etfoglasiserver.repository.SubjectRepo;
 import dev.ksan.etfoglasiserver.repository.UserRepo;
 
 import java.util.List;
@@ -19,8 +20,6 @@ import org.springframework.security.authentication.AuthenticationManager;
 import org.springframework.security.authentication.BadCredentialsException;
 import org.springframework.security.authentication.UsernamePasswordAuthenticationToken;
 import org.springframework.security.core.Authentication;
-import org.springframework.security.core.userdetails.UserDetails;
-import org.springframework.security.core.userdetails.UsernameNotFoundException;
 import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
@@ -28,12 +27,18 @@ import org.springframework.transaction.annotation.Transactional;
 @Service
 @Transactional
 public class UserService {
+    @Autowired
+    SubjectRepo subjectRepo;
+
   @Autowired UserRepo userRepo;
 
   @Autowired private AuthenticationManager authManager;
 
   @Autowired private JWTService jwtService;
 
+
+
+
   private BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();
 
   public List<UserDTO> getUsers() {
@@ -70,10 +75,10 @@ public class UserService {
         if (user.getPassword() != null && user.getPassword().length() > 0) {
 
           if (this.isValidPassword(user.getPassword())) {
-            existingUser.setPassword(user.getPassword());
+            existingUser.setPassword(encoder.encode(user.getPassword()));
           } else throw new RuntimeException("Password too short");
         }
-        if (user.getNewEmail() != null && user.getNewEmail() != existingUser.getEmail()) {
+        if (user.getNewEmail() != null && !user.getNewEmail().equals(existingUser.getEmail())) {
 
           existingUser.setEmail(user.getNewEmail());
         } else {
@@ -83,8 +88,6 @@ public class UserService {
 
         existingUser.setSubjectSet(user.getSubjectSet());
 
-        existingUser.setNotificationMethod(user.getNotificationMethod());
-
         userRepo.save(existingUser);
 
         return getUserDTO(existingUser);
@@ -141,17 +144,25 @@ public class UserService {
 
     public JwtResponseDTO verify(UserLoginDTO user) {
 
+        Authentication authentication = authManager.authenticate(
+                new UsernamePasswordAuthenticationToken(user.getEmail(), user.getPassword())
+        );
 
-    Authentication authentication = authManager.authenticate(new UsernamePasswordAuthenticationToken(user.getEmail(), user.getPassword()));
-
-    if (authentication.isAuthenticated()) {
-
-      String token = jwtService.generateToken(user.getEmail());
-      return new JwtResponseDTO(token, user.getEmail(), HttpStatus.OK.toString());
-    } else {
-
+        if (!authentication.isAuthenticated()) {
             throw new BadCredentialsException("Invalid username or password");
         }
+
+        String token = jwtService.generateToken(user.getEmail());
+
+        User foundUser = userRepo.findByEmail(user.getEmail())
+                .orElseThrow(() -> new RuntimeException("User not found after authentication"));
+
+        return new JwtResponseDTO(
+                token,
+                foundUser.getEmail(),
+                foundUser.getId().toString(),
+                HttpStatus.OK.toString()
+        );
     }
 
   public Optional<User> getAccountByEmail(String email) {
@@ -159,4 +170,34 @@ public class UserService {
   }
 
 
+    public UserDTO addSubject(UUID userId, UUID subjectId) {
+
+        User user = userRepo.findById(userId)
+                .orElseThrow(() -> new RuntimeException("User not found"));
+
+        Subject subject = subjectRepo.findById(subjectId)
+                .orElseThrow(() -> new RuntimeException("Subject not found"));
+
+        user.getSubjectSet().add(subject);
+
+        userRepo.save(user);
+
+        return getUserDTO(user);
+    }
+
+    public UserDTO removeSubject(UUID userId, UUID subjectId) {
+
+        User user = userRepo.findById(userId)
+                .orElseThrow(() -> new RuntimeException("User not found"));
+
+        Subject subject = subjectRepo.findById(subjectId)
+                .orElseThrow(() -> new RuntimeException("Subject not found"));
+
+
+        user.getSubjectSet().remove(subject);
+
+        userRepo.save(user);
+
+        return getUserDTO(user);
+    }
 }