[레벨 2] 07. 1단계 - 장바구니 - 협업 미션

@Component
public class JwtTokenProvider {

    private final SecretKey key;
    private final long validityInMilliseconds;

    public JwtTokenProvider(@Value("${security.jwt.token.secret-key}") String secretKey,
                            @Value("${security.jwt.token.expire-length}") long validityInMilliseconds) {
        this.key = Keys.hmacShaKeyFor(secretKey.getBytes(StandardCharsets.UTF_8));
        this.validityInMilliseconds = validityInMilliseconds;
    }

    public String createToken(String payload) {
        Date now = new Date();
        Date validity = new Date(now.getTime() + validityInMilliseconds);

        return Jwts.builder()
                .setSubject(payload)
                .setIssuedAt(now)
                .setExpiration(validity)
                .signWith(key, SignatureAlgorithm.HS256)
                .compact();
    }

    public String getPayload(String token) {
        return Jwts.parserBuilder()
                .setSigningKey(key)
                .build()
                .parseClaimsJws(token)
                .getBody()
                .getSubject();
    }

    public void validateToken(String token) {
        try {
            Jws<Claims> claims = Jwts.parserBuilder()
                    .setSigningKey(key)
                    .build()
                    .parseClaimsJws(token);

            claims.getBody().getExpiration().before(new Date());
        } catch (JwtException | IllegalArgumentException e) {
            throw new InvalidTokenException();
        }
    }
}

public class AuthenticationPrincipalArgumentResolver implements HandlerMethodArgumentResolver {

    private static final String AUTHORIZATION_HEADER_NAME = "Authorization";

    private final JwtTokenProvider jwtTokenProvider;

    public AuthenticationPrincipalArgumentResolver(JwtTokenProvider jwtTokenProvider) {
        this.jwtTokenProvider = jwtTokenProvider;
    }

    @Override
    public boolean supportsParameter(MethodParameter parameter) {
        return parameter.hasParameterAnnotation(AuthenticationPrincipal.class);
    }

    @Override
    public Object resolveArgument(MethodParameter parameter, ModelAndViewContainer mavContainer,
                                  NativeWebRequest webRequest, WebDataBinderFactory binderFactory) {
        String authorizationHeader = webRequest.getHeader(AUTHORIZATION_HEADER_NAME);
        String accessToken = AuthorizationExtractor.extract(authorizationHeader);

        if (!jwtTokenProvider.validateToken(accessToken)) {
            throw new InvalidTokenException();
        }

        String username = jwtTokenProvider.getPayload(accessToken);
        return new LoginCustomer(username);
    }
}

@RestController
@RequestMapping("/api/customers")
public class CustomerController {
    ...
    @GetMapping("/me")
    public ResponseEntity<CustomerResponse> findCustomer(@AuthenticationPrincipal LoginCustomer loginCustomer) {
        CustomerResponse customerResponse = customerService.find(loginCustomer);
        return ResponseEntity.ok(customerResponse);
    }
    ...
}

@Configuration
public class AuthenticationPrincipalConfig implements WebMvcConfigurer {

    private final JwtTokenProvider jwtTokenProvider;

    public AuthenticationPrincipalConfig(JwtTokenProvider jwtTokenProvider) {
        this.jwtTokenProvider = jwtTokenProvider;
    }

    @Override
    public void addArgumentResolvers(List<HandlerMethodArgumentResolver> argumentResolvers) {
        argumentResolvers.add(createAuthenticationPrincipalArgumentResolver());
    }

    @Bean
    public HandlerMethodArgumentResolver createAuthenticationPrincipalArgumentResolver() {
        return new AuthenticationPrincipalArgumentResolver();
    }
    ...
}

public class AuthenticationPrincipalArgumentResolver implements HandlerMethodArgumentResolver {
    ...
    private static final String AUTHORIZATION_HEADER_NAME = "Authorization";
    ...
}

`리뷰어`: Http 명세에 정의된 헤더는 `org.springframework.http.HttpHeaders`에 상수로 정의되어있어요. 이걸 활용해도 좋을거 같네요 :)

public class InvalidTokenFormatException extends RuntimeException {

    public InvalidTokenFormatException() {
        this("token 형식이 잘못 되었습니다. (형식: Bearer aaaaaaaa.bbbbbbbb.cccccccc)");
    }
}

`리뷰어`: 위 메세지는 Advice를 통해서 클라이언트에 응답으로 내려가는 메세지로 보이네요. 혹시 화면에서 위 메세지가 보이는걸까요? 그렇다면 유저 친화적인 메세지로 변경해보면 어떨까요? 트레킹이 필요하다면 로깅을 활용해보면 좋을거 같아요.

`매트`: 해당 예외는 고객에게 전달하기 위한 목적보다 프론트 개발자를 위해 전달하기 위한 목적이었습니다! 프론트 개발자에게 이러한 예외 상황을 전달하기 위해서는 로깅을 활용하는 방법이 선호되어 추천해주신 건가요?!

`리뷰어`: 프론트 개발자에게 에러 상황을 알려주기 위함이군요. 매트가 위 Exception의 메세지를 가지고 트레킹 하는걸로 생각했어서 로깅을 말씀드린거긴해요. 지금 상황에서는 위 응답의 메세지를 프론트에서 사용하지 않도록 가이드가 되어야 할 것 같은데요. 서버에서 에러 상황에 대한 유저 메세지를 제어할 필요는 없을지도 고민해보면 좋을것 같습니다.

`리뷰어`: 수정, 삭제시 200 응답도 고려해볼 수 있지 않을까 싶은데요. 204를 선택하신 이유가 있을까요?

`매트`: 리소스를 변경하는 행위와 조회하는 행위를 분리하기 위한 목적입니다! 수정과 동시에 조회를 통해 body에 해당 데이터를 담을 수도 있겠지만 한 가지 요청에 너무 많은 책임을 가지고 있다고 생각하여 수정 및 삭제가 완료 되었다는 것을 명시적으로 표현하기 위해204를 활용하였습니다!

`리뷰어`: 프론트 화면이 어떻게 되는지, 어떻게 협의가 되었는지 모르겠지만 수정시에 수정된 결과나 삭제시 삭제한 엔티티의 아이디를 내려주는 경우도 종종있어요. 수정시에 내려준 결과 id나 삭제 id로 프론트 단에서 처리하는 경우에 종종 그렇게 사용하는데요. 이렇게 하면 서버 단에서도 조회하는 API 콜 하나를 줄여줄 수도 있어서 필요하다면 고려해보셔도 괜찮을거 같습니다.