From abbfde07564b25f416f527085100191bad7c96ab Mon Sep 17 00:00:00 2001
From: Anthony Stirling <77850077+Frooodle@users.noreply.github.com>
Date: Sun, 9 Nov 2025 18:34:24 +0000
Subject: [PATCH] missed class from previous merge

---
 .../pdfjson/JobOwnershipServiceImpl.java      | 22 ++++++++++++-------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/app/proprietary/src/main/java/stirling/software/SPDF/service/pdfjson/JobOwnershipServiceImpl.java b/app/proprietary/src/main/java/stirling/software/SPDF/service/pdfjson/JobOwnershipServiceImpl.java
index 6c6213c69..8fb6814f3 100644
--- a/app/proprietary/src/main/java/stirling/software/SPDF/service/pdfjson/JobOwnershipServiceImpl.java
+++ b/app/proprietary/src/main/java/stirling/software/SPDF/service/pdfjson/JobOwnershipServiceImpl.java
@@ -2,13 +2,14 @@ package stirling.software.SPDF.service.pdfjson;
 
 import java.util.Optional;
 
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
-import org.springframework.security.core.Authentication;
-import org.springframework.security.core.context.SecurityContextHolder;
 import org.springframework.stereotype.Service;
 
 import lombok.extern.slf4j.Slf4j;
 
+import stirling.software.common.service.UserServiceInterface;
+
 /**
  * Service to manage job ownership and access control for PDF JSON operations. When security is
  * enabled, jobs are scoped to authenticated users. When security is disabled, jobs are globally
@@ -20,23 +21,28 @@ import lombok.extern.slf4j.Slf4j;
 public class JobOwnershipServiceImpl
         implements stirling.software.common.service.JobOwnershipService {
 
+    @Autowired(required = false)
+    private UserServiceInterface userService;
+
     /**
      * Get the current authenticated user's identifier. Returns empty if no user is authenticated.
      *
      * @return Optional containing user identifier, or empty if not authenticated
      */
     public Optional<String> getCurrentUserId() {
+        if (userService == null) {
+            log.debug("UserService not available");
+            return Optional.empty();
+        }
+
         try {
-            Authentication authentication = SecurityContextHolder.getContext().getAuthentication();
-            if (authentication != null
-                    && authentication.isAuthenticated()
-                    && !"anonymousUser".equals(authentication.getPrincipal())) {
-                String username = authentication.getName();
+            String username = userService.getCurrentUsername();
+            if (username != null && !username.isEmpty() && !"anonymousUser".equals(username)) {
                 log.debug("Current authenticated user: {}", username);
                 return Optional.of(username);
             }
         } catch (Exception e) {
-            log.warn("Failed to get current user from security context: {}", e.getMessage());
+            log.warn("Failed to get current username from UserService: {}", e.getMessage());
         }
         return Optional.empty();
     }