Skip to content

Commit 528af23

Browse files
committed
SEBSERV-885 better logging
1 parent f580375 commit 528af23

2 files changed

Lines changed: 6 additions & 0 deletions

File tree

src/main/java/ch/ethz/seb/sps/server/datalayer/dao/impl/SessionDAOBatis.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -513,6 +513,9 @@ public Result<EntityKey> secureDeleteSession(final String sessionUUID) {
513513
.execute();
514514

515515
// secure delete all images
516+
if (log.isDebugEnabled()) {
517+
log.debug("Secure delete all Screenshots for session {}, screenshots: {}", sessionUUID, screenShotPKs);
518+
}
516519
this.screenshotDAO
517520
.secureDeleteAllForSession(sessionUUID, screenShotPKs)
518521
.getOrThrow();
@@ -531,6 +534,7 @@ public Result<EntityKey> secureDeleteSession(final String sessionUUID) {
531534

532535
sessionRecordMapper.deleteByPrimaryKey(sessionPK);
533536

537+
log.info("Successfully and securely delete user session data: {}", sessionUUID);
534538
return new EntityKey(sessionUUID, EntityType.SESSION);
535539
});
536540
}

src/main/java/ch/ethz/seb/sps/server/weblayer/AdminSessionController.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,8 @@ protected Session createNew(final POSTMapper postParams) {
169169
produces = MediaType.APPLICATION_JSON_VALUE)
170170
public Collection<EntityKey> secureDelete(@PathVariable final String modelId) {
171171

172+
log.info("Got secure session deletion request for session: {}", modelId);
173+
172174
return this.entityDAO.byModelId(modelId)
173175
.flatMap(this::checkWriteAccess)
174176
.flatMap(this::validForDelete)

0 commit comments

Comments
 (0)