Improve user metadata handling in OktaOAuthAuthenticationService

Refactor OktaOAuthAuthenticationService.java to enhance handling of user metadata during login process. The old deserialization method of metadata is replaced with a more efficient one, enabling descriptive logging. A JsonProcessingException is now handled by throwing a runtime exception.

pic-sure-auth-services/src/main/java/edu/harvard/hms/dbmi/avillach/auth/service/auth/OktaOAuthAuthenticationService.java

@@ -1,5 +1,6 @@
 package edu.harvard.hms.dbmi.avillach.auth.service.auth;
 
+import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.node.ObjectNode;
 import edu.harvard.dbmi.avillach.util.HttpClientUtil;
@@ -133,9 +134,10 @@ private User loadUser(JsonNode introspectResponse) {
             // Add metadata to the user upon logging in if it doesn't exist
             if (StringUtils.isBlank(user.getGeneralMetadata())) {
                 ObjectNode objectNode = generateUserMetadata(introspectResponse, user);
-                logger.info("Adding metadata to user: " + user.getUuid() + " ___ " + objectNode.asText());
+                String userMetadata = JAXRSConfiguration.objectMapper.writeValueAsString(objectNode);
+                logger.info("Adding metadata to user: " + user.getUuid() + " ___ " + userMetadata);
                 // Set the general metadata to the objectNode
-                user.setGeneralMetadata(objectNode.asText());
+                user.setGeneralMetadata(userMetadata);
             }
 
             userRepository.persist(user);
@@ -144,6 +146,8 @@ private User loadUser(JsonNode introspectResponse) {
         } catch (NoResultException ex) {
             logger.info("LOGIN FAILED ___ USER NOT FOUND ___ " + userEmail + " ___");
             return null;
+        } catch (JsonProcessingException e) {
+            throw new RuntimeException(e);
         }
     }
 
@@ -166,6 +170,7 @@ private ObjectNode generateUserMetadata(JsonNode introspectResponse, User user)
         authzNode.put("user_id", user.getUuid().toString());
         authzNode.put("username", user.getEmail());
         tagsNode.put("email", user.getEmail());
+
         return objectNode;
     }