pricelees/roomescape-refactored
1
0
Fork 0
generated from pricelees/issue-pr-template
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: AuthController에서의 쿠키 관련 로직 제거 및 ResponseEntity 적용

Browse Source
This commit is contained in:
이상진 2025-07-15 11:25:45 +09:00
parent 651557ca87
commit 8be20f2e7c
1 changed files with 17 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

45
src/main/java/roomescape/auth/web/AuthController.kt
View File

@ -1,21 +1,19 @@
package roomescape.auth.web
import io.swagger.v3.oas.annotations.Parameter
import jakarta.servlet.http.Cookie
import jakarta.servlet.http.HttpServletRequest
import jakarta.servlet.http.HttpServletResponse
import jakarta.validation.Valid
import org.springframework.http.HttpHeaders
import org.springframework.http.ResponseEntity
import org.springframework.web.bind.annotation.GetMapping
import org.springframework.web.bind.annotation.PostMapping
import org.springframework.web.bind.annotation.RequestBody
import org.springframework.web.bind.annotation.RestController
import roomescape.auth.docs.AuthAPI
import roomescape.auth.service.AuthService
import roomescape.auth.web.support.MemberId
import roomescape.auth.web.support.accessTokenCookie
import roomescape.auth.web.support.addAccessTokenCookie
import roomescape.auth.web.support.expire
import roomescape.auth.web.support.toCookie
import roomescape.common.dto.response.RoomescapeApiResponse
import roomescape.auth.web.support.expiredAccessTokenCookie
import roomescape.auth.web.support.toResponseCookie
import roomescape.common.dto.response.CommonApiResponse
@RestController
class AuthController(
@ -25,34 +23,25 @@ class AuthController(
@PostMapping("/login")
override fun login(
@Valid @RequestBody loginRequest: LoginRequest,
response: HttpServletResponse
): RoomescapeApiResponse<Void> {
val accessToken: TokenResponse = authService.login(loginRequest)
val cookie: Cookie = accessToken.toCookie()
): ResponseEntity<CommonApiResponse<Unit>> {
val response: TokenResponse = authService.login(loginRequest)
response.addAccessTokenCookie(cookie)
return RoomescapeApiResponse.success()
return ResponseEntity.ok()
.header(HttpHeaders.SET_COOKIE, response.toResponseCookie())
.body(CommonApiResponse())
}
@GetMapping("/login/check")
override fun checkLogin(
@MemberId @Parameter(hidden = true) memberId: Long
): RoomescapeApiResponse<LoginCheckResponse> {
val response = authService.checkLogin(memberId)
): ResponseEntity<CommonApiResponse<LoginCheckResponse>> {
val response: LoginCheckResponse = authService.checkLogin(memberId)
return RoomescapeApiResponse.success(response)
return ResponseEntity.ok(CommonApiResponse(response))
}
@PostMapping("/logout")
override fun logout(
request: HttpServletRequest,
response: HttpServletResponse
): RoomescapeApiResponse<Void> {
val cookie: Cookie = request.accessTokenCookie()
cookie.expire()
response.addAccessTokenCookie(cookie)
return RoomescapeApiResponse.success()
}
override fun logout(): ResponseEntity<CommonApiResponse<Unit>> = ResponseEntity.ok()
.header(HttpHeaders.SET_COOKIE, expiredAccessTokenCookie())
.body(CommonApiResponse())
}