woowacourse-teams / 2022-f12

개발자의 모든 도구, F12 💻

https://f12.app

82 stars 7 forks source link

[Refactor/BE] 불필요한 ResponseEntity 제거 #878

Closed Ohzzi closed 1 year ago

Ohzzi commented 1 year ago

issue: closes #863

작업 내용

200 OK를 내려주는 응답에 대해서 스프링은 ResponseEntity의 형태가 아니라 T의 반환값으로도 처리할 수 있음.
특별히 처리를 해주어야 하는 메서드가 없었기 때문에 모든 200 OK 응답들은 ResponseBody로 사용할 DTO를 그대로 리턴값으로 사용하도록 수정

의문점

Cookie 값을 내려주는 컨트롤러 메서드에서 왜 굳이 HttpResponse를 파라미터로 받아서 처리했을까?
```
return ResponseEntity.ok().header()...
```
으로도 처리할 수 있지 않았을까?