improve logging

dataquest-dev · Dec 23, 2024 · e04ae1b · e04ae1b
1 parent 6bd2e76
commit e04ae1b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/dspace-api/src/main/java/org/dspace/core/Context.java b/dspace-api/src/main/java/org/dspace/core/Context.java
@@ -475,6 +475,7 @@ public void commit() throws SQLException {
                 // Commit our changes (this closes the transaction but leaves database connection open)
                 dbConnection.commit();
                 reloadContextBoundEntities();
+                log.info("commit() method - Committing transaction END.");
             } else {
                 log.info("commit() method - No database connection to commit to.");
             }
@@ -602,6 +603,7 @@ public void rollback() throws SQLException {
                 dbConnection.rollback();
                 log.info("rollback() method - Transaction successfully rolled back.");
                 reloadContextBoundEntities();
+                log.info("rollback() method - Transaction successfully rolled back END.");
             }
         } catch (SQLException e) {
             log.error("Error rolling back transaction", e);
@@ -812,6 +814,7 @@ protected void finalize() throws Throwable {
 
         log.info("finalize() method - calling super.finalize().");
         super.finalize();
+        log.info("finalize() method - calling super.finalize() END.");
     }
 
     public void shutDownDatabase() throws SQLException {

diff --git a/dspace-api/src/main/java/org/dspace/eperson/EPerson.java b/dspace-api/src/main/java/org/dspace/eperson/EPerson.java
@@ -125,7 +125,6 @@ public class EPerson extends DSpaceObject implements DSpaceObjectLegacySupport {
      * {@link org.dspace.eperson.service.EPersonService#create(Context)}
      */
     protected EPerson() {
-        ThreadContext.put("classID", String.valueOf(System.identityHashCode(this)));
         Exception exception = new Exception("Stack trace for EPerson creation");
         log.info("EPerson created", exception);
     }

diff --git a/...src/main/java/org/dspace/app/rest/security/jwt/JWTTokenRestAuthenticationServiceImpl.java b/...src/main/java/org/dspace/app/rest/security/jwt/JWTTokenRestAuthenticationServiceImpl.java
@@ -262,6 +262,7 @@ private void addTokenToResponse(final HttpServletRequest request, final HttpServ
             resetCSRFToken(request, response);
         }
         response.setHeader(AUTHORIZATION_HEADER, String.format("%s %s", AUTHORIZATION_TYPE, token));
+        log.info("DQ: Added token to request");
     }
 
     /**
-Original file line number
+Diff line change
@@ Expand Up @@
                 resetCSRFToken(request, response);
             }
             response.setHeader(AUTHORIZATION_HEADER, String.format("%s %s", AUTHORIZATION_TYPE, token));
+            log.info("DQ: Added token to request");
         }
         /**
@@ Expand Down @@