Skip to content

Commit

Permalink
🚑 fix: Add cookie domain
Browse files Browse the repository at this point in the history
- 쿠키 도메인 정보 추가

Related: #42
  • Loading branch information
gmkim20713 committed Dec 23, 2023
1 parent 36503e5 commit 384e2b3
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
import jakarta.servlet.http.HttpServletRequest;
import jakarta.validation.Valid;
import lombok.RequiredArgsConstructor;
import org.springframework.beans.factory.annotation.Value;
import org.springframework.boot.web.server.Cookie;
import org.springframework.http.*;
import org.springframework.web.bind.annotation.*;
Expand All @@ -18,6 +19,9 @@
@RequiredArgsConstructor
public class AuthController {

@Value("${domain}")
private String domain;

private final long COOKIE_EXPIRATION = 7776000; // 90일

private final AuthService authService;
Expand All @@ -38,6 +42,7 @@ public ResponseEntity<?> login(@RequestBody LoginRequestDto loginRequest) {
HttpCookie httpCookie = ResponseCookie.from("refresh-token", tokenDto.getRefreshToken())
.maxAge(COOKIE_EXPIRATION)
.path("/")
.domain(domain)
.httpOnly(true)
.secure(true)
.sameSite(Cookie.SameSite.NONE.attributeValue()) //서드파티 쿠키 사용 허용
Expand Down Expand Up @@ -76,6 +81,7 @@ public ResponseEntity<?> reissue(@CookieValue("refresh-token") String requestRef
ResponseCookie responseCookie = ResponseCookie.from("refresh-token", newAuthToken.getRefreshToken())
.maxAge(COOKIE_EXPIRATION)
.path("/")
.domain(domain)
.httpOnly(true)
.secure(true)
.sameSite(Cookie.SameSite.NONE.attributeValue())
Expand Down
4 changes: 3 additions & 1 deletion src/main/resources/application.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,4 +42,6 @@ server:

springdoc:
swagger-ui:
path: /swagger-vote.html
path: /swagger-vote.html

domain: ENC(wtMoW4E5IOZgq215uWHzGsBlrqecbyNr9lsrql5da0LFtz4cywsN8Fap3bhNqVYQ0RuvblLIewQZo+u8pBCaFA==)

0 comments on commit 384e2b3

Please sign in to comment.