Skip to content

Commit

Permalink
로그 줄이기, 변수명 수정
Browse files Browse the repository at this point in the history
  • Loading branch information
@sukangpunch
sukangpunch committed Jul 20, 2024
1 parent 3e6474e commit 48fdab9
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 28 deletions.
14 changes: 0 additions & 14 deletions src/main/java/me/snaptime/jwt/JwtProvider.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -33,14 +33,10 @@ public class JwtProvider {

@PostConstruct
protected void init(){
log.info("[init] JwtTokenProvide 내 secretKey 초기화 시작");
secretKey = Keys.secretKeyFor(SignatureAlgorithm.HS256);
log.info("[init] JwtTokenProvider 내 secretKey 초기화 완료");
}

public String createAccessToken(Long userId, String loginId, List<String> roles){
log.info("[createAccessToken] 엑세스 토큰 생성 시작");

Claims claims = Jwts.claims().setSubject(loginId);
claims.put("userId",userId);
claims.put("type","access");
Expand All @@ -53,13 +49,10 @@ public String createAccessToken(Long userId, String loginId, List<String> roles)
.signWith(secretKey)
.compact();

log.info("[createAccessToken] 엑세스 토큰 생성 완료");
return token;
}

public String createRefreshToken(Long id, String loginId, List<String> roles){
log.info("[createRefreshToken] 리프레시 토큰 생성 시작");

Claims claims = Jwts.claims().setSubject(loginId);
claims.put("userId", id);
claims.put("type", "refresh");
Expand All @@ -71,13 +64,10 @@ public String createRefreshToken(Long id, String loginId, List<String> roles){
.setExpiration(new Date(now.getTime() + refreshTokenValidTime))
.signWith(secretKey)
.compact();

log.info("[createAccessToken] 엑세스 토큰 생성 완료");
return token;
}

public Long getUserId(String token) {
log.info("[getUserId] 토큰 기반 회원 구별 정보 추출");
Long userId = Long.valueOf(Jwts.parser().setSigningKey(secretKey).parseClaimsJws(token).getBody().get("userId").toString());
log.info("[getUserId] 토큰 기반 회원 구별 정보 추출 완료, userId : {}", userId);
return userId;
Expand All @@ -86,7 +76,6 @@ public Long getUserId(String token) {
// 필터에서 인증 성공 후, SecurityContextHolder 에 저장할 Authentication 을 생성
//UsernamePasswordAuthenticationToken 클래스를 사용
public Authentication getAuthentication(String token){
log.info("[getAuthentication] 토큰 인증 정보 조회 시작");
UserDetails userDetails = userDetailsService.loadUserByUsername(this.getUsername(token));
log.info("[getAuthentication] 토큰 인증 정보 조회 완료, UserDetails loginId : {}",userDetails.getUsername());

Expand All @@ -97,7 +86,6 @@ public Authentication getAuthentication(String token){
//Jwts.parser()를 통해 secretKey를 설정하고 클레임을 추출해서 토큰을 생성할 때 넣었던 sub값을 추출합니다.
public String getUsername(String token)
{
log.info("[getUsername] 토큰 기반 회원 구별 정보 추출");
String loginId = Jwts.parserBuilder()
.setSigningKey(secretKey)
.build()
Expand All @@ -109,7 +97,6 @@ public String getUsername(String token)
}

public String getAuthorizationToken(HttpServletRequest request){
log.info("[getAuthorizationToken] HTTP 헤더에서 Token 값 추출");
String token = request.getHeader("Authorization");
try{
if(!token.substring(0,"BEARER ".length()).equalsIgnoreCase("Bearer ")){
Expand All @@ -126,7 +113,6 @@ public String getAuthorizationToken(HttpServletRequest request){
이 메소드는 토큰을 전달 받아 클레임의 유효기간을 체크하고 boolean 타입 값을 리턴하는 역할을 한다.
*/
public boolean validateToken(String token) {
log.info("[validateToken] 토큰 유효 체크 시작");
try{
//복잡한 설정일 떈, Jwts.parserBuilder()를 이용
Jws<Claims> claims = Jwts.parser()
Expand Down
8 changes: 4 additions & 4 deletions src/main/java/me/snaptime/profile/controller/ProfileController.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ public class ProfileController {
"<br> 다른 사람의 프로필 조회 -> snap이 전부 private이거나 없는 경우 앨범 리턴 x 그리고 private 인 snap 리턴 x")
@Parameter(name = "targetLoginId", description = "앨범과 사진들을 가져오기 위한 loginId", required = true)
@GetMapping("/album-snap")
public ResponseEntity<CommonResponseDto<List<AlbumSnapResDto>>> getAlbumSnap(@AuthenticationPrincipal UserDetails principal,
public ResponseEntity<CommonResponseDto<List<AlbumSnapResDto>>> getAlbumSnap(@AuthenticationPrincipal UserDetails userDetails,
@RequestParam("targetLoginId")
@NotBlank(message = "로그인 아이디 입력은 필수입니다.") String targetLoginId){
String reqLoginId = principal.getUsername();
String reqLoginId = userDetails.getUsername();
List<AlbumSnapResDto> albumSnapResDtos = profileService.getAlbumSnap(reqLoginId, targetLoginId);
return ResponseEntity.status(HttpStatus.OK).body(
new CommonResponseDto<>(
Expand All @@ -57,10 +57,10 @@ public ResponseEntity<CommonResponseDto<List<AlbumSnapResDto>>> getAlbumSnap(@Au
"<br> 토큰이 없어도 해당 Api 엔드포인트를 요청할 수 있습니다.")
@Parameter(name = "targetLoginId", description = "이름과 프로필 사진을 가져오기 위한 loginId", required = true)
@GetMapping("/profile")
public ResponseEntity<CommonResponseDto<UserProfileResDto>> getUserProfile(@AuthenticationPrincipal UserDetails principal,
public ResponseEntity<CommonResponseDto<UserProfileResDto>> getUserProfile(@AuthenticationPrincipal UserDetails userDetails,
@RequestParam("targetLoginId")
@NotBlank(message = "로그인 아이디 입력은 필수입니다.") String targetLoginId){
String reqLoginId = principal.getUsername();
String reqLoginId = userDetails.getUsername();
UserProfileResDto userProfileResDto = profileService.getUserProfile(reqLoginId, targetLoginId);
return ResponseEntity.status(HttpStatus.OK).body(
new CommonResponseDto<>(
Expand Down
4 changes: 2 additions & 2 deletions src/main/java/me/snaptime/profilePhoto/controller/ProfilePhotoController.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,9 @@ public ResponseEntity<?> downloadProfileToFileSystem(@PathVariable("profilePhoto
@Operation(summary = "프로필 사진 수정",description = "유저의 프로필 사진을 수정 합니다." +
"<br> 유저의 Token 으로 유저를 구분짓습니다.")
@PutMapping(consumes = MULTIPART_FORM_DATA)
public ResponseEntity<?> updateProfileToFileSystem(@AuthenticationPrincipal UserDetails principal,
public ResponseEntity<?> updateProfileToFileSystem(@AuthenticationPrincipal UserDetails userDetails,
@RequestParam MultipartFile file) throws Exception {
String loginId = principal.getUsername();
String loginId = userDetails.getUsername();
log.info("[updateProfile] 유저의 프로필 사진을 수정합니다. loginId : {}", loginId);
ProfilePhotoResDto updateProfile = profilePhotoService.updatePhotoFromFileSystem(loginId, file);
return ResponseEntity.status(HttpStatus.OK).body(
Expand Down
16 changes: 8 additions & 8 deletions src/main/java/me/snaptime/user/controller/UserController.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,8 @@ public class UserController {

@Operation(summary = "유저 정보 조회",description = "유저 번호로 유저 정보를 조회합니다. ")
@GetMapping()
public ResponseEntity<CommonResponseDto<UserResDto>> getUser(@AuthenticationPrincipal UserDetails principal){
UserResDto userResDto = userService.getUser(principal.getUsername());
public ResponseEntity<CommonResponseDto<UserResDto>> getUser(@AuthenticationPrincipal UserDetails userDetails){
UserResDto userResDto = userService.getUser(userDetails.getUsername());
return ResponseEntity.status(HttpStatus.OK).body(
new CommonResponseDto<>(
"유저 정보가 성공적으로 조회되었습니다.",
Expand All @@ -48,20 +48,20 @@ public ResponseEntity<CommonResponseDto<UserResDto>> getUser(@AuthenticationPrin
"<br> 유저 loginId 수정 이후에는, Token의 loginId 정보와 현재 유저의 loginId가 다르므로," +
"<br> Token을 버리고 재 login을 유도해야 합니다.")
@PatchMapping()
public ResponseEntity<CommonResponseDto<UserResDto>> changeUser(@AuthenticationPrincipal UserDetails principal,
public ResponseEntity<CommonResponseDto<UserResDto>> changeUser(@AuthenticationPrincipal UserDetails userDetails,
@Valid @RequestBody UserUpdateReqDto userUpdateReqDto){
UserResDto userResDto = userService.updateUser(principal.getUsername(), userUpdateReqDto);
UserResDto userResDto = userService.updateUser(userDetails.getUsername(), userUpdateReqDto);
return ResponseEntity.status(HttpStatus.OK).body(
new CommonResponseDto<>(
"유저 정보 수정이 성공적으로 완료되었습니다.",
userResDto));
}
@Operation(summary = "유저 비밀번호 수정",description = "해당 유저의 비밀번호를 수정합니다.")
@PatchMapping("/password")
public ResponseEntity<CommonResponseDto<Void>> changeUser(@AuthenticationPrincipal UserDetails principal,
public ResponseEntity<CommonResponseDto<Void>> changeUser(@AuthenticationPrincipal UserDetails userDetails,
@RequestParam("password")
@NotBlank(message = "로그인 아이디 입력은 필수입니다.") String password) {
userService.updatePassword(principal.getUsername(), password);
userService.updatePassword(userDetails.getUsername(), password);
return ResponseEntity.status(HttpStatus.OK).body(
new CommonResponseDto<>(
"유저 비밀번호 수정이 성공적으로 완료되었습니다.",
Expand All @@ -71,8 +71,8 @@ public ResponseEntity<CommonResponseDto<Void>> changeUser(@AuthenticationPrincip

@Operation(summary = "유저 삭제",description = "유저 번호로 유저를 삭제합니다.")
@DeleteMapping()
public ResponseEntity<CommonResponseDto<Void>> deleteUser(@AuthenticationPrincipal UserDetails principal){
userService.deleteUser(principal.getUsername());
public ResponseEntity<CommonResponseDto<Void>> deleteUser(@AuthenticationPrincipal UserDetails userDetails){
userService.deleteUser(userDetails.getUsername());
return ResponseEntity.status(HttpStatus.OK).body(
new CommonResponseDto<>(
"유저 삭제가 성공적으로 완료되었습니다.",
Expand Down

0 comments on commit 48fdab9

Please sign in to comment.