trivial:유의미한 데이터를 log 로 남기기 (#338)

* group 수정

* notice log

* problem log 수정

* solution log 수정

* user log 수정

* trivial:user,problem log 수정

* trivial : noti log 수정

* trivial:notice log 수정

* trivial:group log 수정

* trivial:merge develop branch

Author: s-hwan

src/main/java/com/gamzabat/algohub/feature/group/studygroup/service/StudyGroupService.java

@@ -121,7 +121,7 @@ public GroupCodeResponse createGroup(User user, CreateGroupRequest request, Mult
 		);
 
 		saveGroupImage(profileImage, group);
-		log.info("success to save study group");
+		log.info("success to save study group user_id={}", user.getId());
 		return new GroupCodeResponse(inviteCode);
 	}
 
@@ -154,7 +154,7 @@ public GetGroupIdResponse joinGroupWithCode(User user, String code) {
 		);
 
 		sendNewMemberNotification(studyGroup, member);
-		log.info("success to join study group");
+		log.info("success to join study group user_id={} code = {} ", user.getId(), code);
 
 		return new GetGroupIdResponse(studyGroup.getId());
 
@@ -175,7 +175,7 @@ public void deleteGroup(User user, Long groupId) {
 
 		deleteAllAboutGroup(group);
 
-		log.info("success to delete study group");
+		log.info("success to delete study group user_id={}, group_id={}", user.getId(), groupId);
 	}
 
 	private void deleteAllAboutGroup(StudyGroup group) {
@@ -216,7 +216,7 @@ public void exitGroup(User user, Long groupId) {
 			}
 		}
 
-		log.info("success to exit study group");
+		log.info("success to exit study group user_id = {} , group_id = {}", user.getId(), groupId);
 	}
 
 	@Transactional
@@ -237,6 +237,7 @@ public void deleteMember(User user, Long userId, Long groupId) {
 					() -> new GroupMemberValidationException(HttpStatus.BAD_REQUEST.value(), "이미 참여하지 않은 회원입니다."));
 
 			studyGroupServiceProvider.getObject().deleteMemberFromStudyGroup(user, groupMember, group);
+			log.info("success to delete member user_id={} , group_id = {}", user.getId(), groupId);
 		} else {
 			throw new UserValidationException("멤버를 삭제 할 권한이 없습니다.");
 		}
@@ -250,7 +251,7 @@ public void deleteMemberFromStudyGroup(User user, GroupMember groupMember, Study
 		solutionRepository.deleteAllByStudyGroupAndUser(studyGroup, user);
 		noticeReadRepository.deleteAllByStudyGroupAndUser(studyGroup, user);
 		groupMemberRepository.delete(groupMember);
-		log.info("success to delete group member");
+		log.info("success to delete group member user_id = {}, group_id = {}", user.getId(), studyGroup.getId());
 	}
 
 	@Transactional(readOnly = true)
@@ -336,7 +337,7 @@ public void editGroup(User user, Long groupId, EditGroupRequest request, Multipa
 		if (request.introduction() != null && !request.introduction().isEmpty())
 			group.editGroupIntroduction(request.introduction());
 
-		log.info("success to edit group info");
+		log.info("success to edit group info user_id={} , group_id = {}", user.getId(), groupId);
 	}
 
 	private void editGroupImage(MultipartFile inputImage, StudyGroup group, Boolean isDefaultImage) {
@@ -534,7 +535,7 @@ public void updateGroupMemberRole(User user, Long groupId, UpdateGroupMemberRole
 		if (RoleOfGroupMember.isOwner(member) && request.role() != null) {
 			owner.updateRole(RoleOfGroupMember.PARTICIPANT);
 		}
-		log.info("success to update group member role");
+		log.info("success to update group member role user_id={} , group_id = {}", user.getId(), group.getId());
 	}
 
 	@Transactional(readOnly = true)
@@ -556,7 +557,7 @@ public void editStudyGroupVisibility(User user, Long groupId, EditGroupVisibilit
 		GroupMember member = groupMemberRepository.findByUserAndStudyGroup(user, group)
 			.orElseThrow(() -> new GroupMemberValidationException(HttpStatus.FORBIDDEN.value(), "참여하지 않은 그룹입니다."));
 		member.updateVisibility(request.isVisible());
-		log.info("success to update group visibility ( userId : {} )", user.getId());
+		log.info("success to update group visibility user_id = {}, group_id = {}", user.getId(), groupId);
 	}
 
 	@Transactional(readOnly = true)

src/main/java/com/gamzabat/algohub/feature/notice/service/NoticeCommentService.java

@@ -48,15 +48,15 @@ public void createComment(User user, Long noticeId, CreateNoticeCommentRequest r
 			.content(request.content())
 			.build());
 
-		log.info("success to create notice comment. commentId: {}, noticeId: {}", comment.getId(), notice.getId());
+		log.info("success to create notice comment. comment_id = {}, notice_id = {}", comment.getId(), notice.getId());
 	}
 
 	@Override
 	@Transactional(readOnly = true)
 	public List<GetCommentResponse> getCommentList(User user, Long noticeId) {
 		Notice notice = validateNotice(user, noticeId);
 		List<NoticeComment> notices = noticeCommentRepository.findAllByNotice(notice);
-		log.info("success to get notice comment list. noticeId: {}", noticeId);
+		log.info("success to get notice comment list. notice_id = {}", noticeId);
 		return notices.stream().map(GetCommentResponse::toDTO).sorted(Comparator.comparing(
 			GetCommentResponse::createdAt).reversed()).toList();
 	}
@@ -70,10 +70,10 @@ public void updateComment(User user, Long commentId, UpdateCommentRequest reques
 		if (!comment.getUser().getId().equals(user.getId()))
 			throw new UserValidationException("댓글 작성자만 수정할 수 있습니다.");
 
-		if(request.content()!=null){
+		if (request.content() != null) {
 			comment.updateComment(request.content());
 		}
-		log.info("success to update notice comment. commentId: {}", comment.getId());
+		log.info("success to update notice comment. comment_id = {}", comment.getId());
 
 	}
 
@@ -88,7 +88,7 @@ public void deleteComment(User user, Long commentId) {
 
 		validateNotice(user, comment.getNotice().getId());
 		noticeCommentRepository.delete(comment);
-		log.info("success to delete notice comment. commentId: {}", commentId);
+		log.info("success to delete notice comment. comment_id = {}", commentId);
 
 	}
 

src/main/java/com/gamzabat/algohub/feature/notice/service/NoticeService.java

@@ -63,7 +63,7 @@ public CreateNoticeResponse createNotice(@AuthedUser User user, Long groupId, Cr
 			.category(request.category())
 			.createdAt(LocalDateTime.now())
 			.build());
-		log.info("success to create notice");
+		log.info("success to create notice user_id={} , group_id = {}", user.getId(), groupId);
 		return new CreateNoticeResponse(notice.getId());
 	}
 
@@ -99,7 +99,7 @@ public void markNoticeAsRead(@AuthedUser User user, Long noticeId) {
 				NoticeRead.builder().notice(notice).user(user).build()
 			);
 		}
-		log.info("success to read notice. userId: {}, noticeId: {}", user.getId(), notice.getId());
+		log.info("success to read notice. user_id = {}, notice_id = {}", user.getId(), notice.getId());
 	}
 
 	@Transactional(readOnly = true)
@@ -132,7 +132,7 @@ public void updateNotice(User user, Long noticeId, UpdateNoticeRequest request)
 		if (request.category() != null) {
 			notice.updateCategory(request.category());
 		}
-		log.info("success to update notice");
+		log.info("success to update notice user_id={} , notice_id = {}", user.getId(), noticeId);
 	}
 
 	@Transactional
@@ -148,7 +148,7 @@ public void deleteNotice(User user, Long noticeId) {
 		noticeReadRepository.deleteAllByNotice(notice);
 		noticeRepository.delete(notice);
 
-		log.info("success to delete notice. userId: {}, noticeId: {}", user.getId(), noticeId);
+		log.info("success to delete notice. user_id = {}, notice_id = {}", user.getId(), noticeId);
 	}
 
 	private void validateStudyGroupExists(Notice notice) {

src/main/java/com/gamzabat/algohub/feature/notification/service/NotificationService.java

@@ -182,7 +182,7 @@ public void readNotification(User user, Long notificationId) {
 		if (!notification.getUser().getId().equals(user.getId()))
 			throw new NotificationValidationException(HttpStatus.FORBIDDEN.value(), "알림의 주인이 일치하지 않습니다.");
 		notification.updateIsRead();
-		log.info("success to read notification. notificationId : {}", notificationId);
+		log.info("success to read notification. notification_id = {}", notificationId);
 	}
 
 	@Transactional
@@ -193,7 +193,7 @@ public void sendNotificationToMembers(StudyGroup group, List<GroupMember> receiv
 		for (GroupMember member : receiver) {
 			NotificationSetting setting = notificationSettingRepository.findByMember(member)
 				.orElseThrow(() -> {
-					log.error("cannot find notification setting for member. userId : {}, groupId : {}",
+					log.error("cannot find notification setting for member. user_id = {}, group_id =g {}",
 						member.getUser().getId(), group.getId());
 					return new CannotFoundNotificationSettingException("해당 그룹에 가입 되지 않은 유저입니다.");
 				});

src/main/java/com/gamzabat/algohub/feature/notification/service/NotificationSettingService.java

@@ -53,6 +53,6 @@ public void editNotificationSettings(User user, EditNotificationSettingRequest r
 			request.deadlineReached()
 		);
 
-		log.info("success to edit notification settings");
+		log.info("success to edit notification settings user_id = {}", user.getId());
 	}
 }

src/main/java/com/gamzabat/algohub/feature/problem/service/ProblemService.java

@@ -91,7 +91,7 @@ public void createProblem(User user, Long groupId, CreateProblemRequest request)
 				NotificationCategory.PROBLEM_STARTED.getMessage(title)
 			);
 
-		log.info("success to create problem");
+		log.info("success to create problem user_id={} , group_id = {}", user.getId(), groupId);
 	}
 
 	@Transactional
@@ -118,7 +118,7 @@ public void editProblem(User user, Long problemId, EditProblemRequest request) {
 			problem.editProblemEndDate(request.endDate());
 		}
 
-		log.info("success to edit problem deadline");
+		log.info("success to edit problem deadline user_id={} , problem_id = {}", user.getId(), problemId);
 	}
 
 	private void checkProblemValidation(Problem problem) {
@@ -212,7 +212,7 @@ public void deleteProblem(User user, Long problemId) {
 
 		solutionRepository.deleteAllByProblem(problem);
 		problemRepository.delete(problem);
-		log.info("success to delete problem");
+		log.info("success to delete problem user_id={} , problem_id = {}", user.getId(), problemId);
 	}
 
 	@Transactional(readOnly = true)

src/main/java/com/gamzabat/algohub/feature/solution/service/SolutionCommentService.java

@@ -57,7 +57,7 @@ public void createComment(User user, Long solutionId, CreateSolutionCommentReque
 			.build());
 
 		sendCommentNotification(user, solution);
-		log.info("success to create solution comment. commentId: {}, solutionId: {}", comment.getId(),
+		log.info("success to create solution comment. comment_id = {}, solution_id = {}", comment.getId(),
 			solution.getId());
 	}
 
@@ -88,7 +88,7 @@ public List<GetCommentResponse> getCommentList(User user, Long solutionId) {
 		}
 		List<GetCommentResponse> result = list.stream().map(GetCommentResponse::toDTO)
 			.sorted((s1, s2) -> s2.createdAt().compareTo(s1.createdAt())).toList();
-		log.info("success to get solution comment list. solutionId: {}", solutionId);
+		log.info("success to get solution comment list. solution_id = {} , user_id = {}", solutionId, user.getId());
 		return result;
 	}
 
@@ -101,7 +101,7 @@ public void updateComment(User user, Long commentId, UpdateCommentRequest reques
 			throw new UserValidationException("댓글 작성자가 아닙니다.");
 
 		comment.updateComment(request.content());
-		log.info("success to update solution comment. commentId: {}", commentId);
+		log.info("success to update solution comment. comment_id = {}, user_id = {}", commentId, user.getId());
 	}
 
 	@Override
@@ -114,7 +114,7 @@ public void deleteComment(User user, Long commentId) {
 
 		checkSolutionValidation(user, comment.getSolution().getId());
 		commentRepository.delete(comment);
-		log.info("success to delete solution comment. commentId: {}", commentId);
+		log.info("success to delete solution comment. comment_id = {}, user_id = {}", commentId, user.getId());
 	}
 
 	private Solution checkSolutionValidation(User user, Long solutionId) {

src/main/java/com/gamzabat/algohub/feature/solution/service/SolutionService.java

@@ -241,6 +241,7 @@ public void createSolution(CreateSolutionRequest request) {
 
 			sendNewSolutionNotification(studyGroup, member.get(), problem);
 		}
+		log.info("success to create solution user_id={}.", user.getId());
 	}
 
 	private void sendNewSolutionNotification(StudyGroup group, GroupMember solver, Problem problem) {

src/main/java/com/gamzabat/algohub/feature/user/service/UserService.java

@@ -92,7 +92,7 @@ public void register(RegisterRequest request, MultipartFile profileImage, String
 			.build());
 
 		saveProfileImage(profileImage, user);
-		log.info("success to register");
+		log.info("success to register user_id={}", user.getId());
 	}
 
 	private void saveProfileImage(MultipartFile profileImage, User user) {
@@ -113,7 +113,7 @@ public TokenResponse signIn(SignInRequest request) {
 			throw new UncorrectedPasswordException("비밀번호가 틀렸습니다.");
 		}
 		JwtDTO result = tokenProvider.generateTokens(authenticate);
-		log.info("success to sign in");
+		log.info("success to sign in identifier = {}", request.identifier());
 		return new TokenResponse(result.getAccessToken(), result.getRefreshToken());
 	}
 
@@ -144,7 +144,7 @@ public void userUpdate(User user, UpdateUserRequest updateUserRequest, Multipart
 		}
 
 		userRepository.save(user);
-		log.info("success to update user");
+		log.info("success to update user user_id={}", user.getId());
 	}
 
 	private void editUserProfileImage(User user, MultipartFile inputImage, Boolean isDefaultImage) {
@@ -179,6 +179,7 @@ public void deleteUser(User user, DeleteUserRequest deleteUserRequest) {
 			throw new UncorrectedPasswordException("비밀번호가 틀렸습니다.");
 		}
 		userRepository.delete(user);
+		log.info("success to delete user user_id={}", user.getId());
 	}
 
 	@Transactional
@@ -199,6 +200,7 @@ public void editPassword(User user, EditUserPasswordRequest request) {
 		user.editPassword(encodedPassword);
 
 		userRepository.save(user);
+		log.info("success to edit password user_id={}", user.getId());
 	}
 
 	@Transactional
@@ -229,6 +231,7 @@ public void deleteBjNickname(User user) {
 	public void checkEmailDuplication(String email) {
 		if (userRepository.existsByEmail(email))
 			throw new UserValidationException("이미 사용 중인 이메일 입니다.");
+		log.info("success to validity email = {}", email);
 	}
 
 	@Transactional(readOnly = true)
@@ -243,7 +246,7 @@ public void checkNickname(String nickname) {
 		if (userRepository.existsByNickname(nickname))
 			throw new CheckNicknameValidationException(HttpStatus.CONFLICT.value(), "이미 사용 중인 닉네임입니다.");
 
-		log.info("success to check nickname validity");
+		log.info("success to check nickname validity nickname={}", nickname);
 	}
 
 	@Transactional(readOnly = true)