[🚌 Issue] dev api 로그인 요청 위치에 따라 redirect url 변환 #245

egg528 · 2024-03-27T14:54:03Z

📒 Issue

[🚌 Issue] dev api 로그인 요청 위치에 따라 redirect url 변환 #244

🎯 어떤 작업을 했는지

state, referer를 key, value로 가지는 LocalStorage 생성
로그인 요청 시 생성된 state값을 key로 referer값 저장
로그인 callback 요청 시 query string으로 넘어온 state값으로 referer값 조회
referer값에 dev-bandiboodi 포함된 경우 dev 프론트 서버로 redirect
단, profile이 dev일 경우만 로직 적용
develop에 적용 후 테스트 필요함

🌍 영향범위 (모듈)

api/application

Resolves: # See also: #

…Map)

#235

binary-ho · 2024-03-27T16:35:36Z

혹시 내일 오전에 리뷰 남겨도 될까요? 쭉 읽긴 했는데 너무 졸려서 머리가 아프네요 ㅠㅠ

egg528 · 2024-03-27T16:38:09Z

혹시 내일 오전에 리뷰 남겨도 될까요? 쭉 읽긴 했는데 너무 졸려서 머리가 아프네요 ㅠㅠ

@binary-ho
천천히 봐주세요~~

binary-ho · 2024-03-27T16:44:59Z

backend/application/api/src/main/kotlin/io/raemian/api/support/security/LoginRedirector.kt

+    val loginRequestRefererStorage: LoginRequestRefererStorage,
+) {
+    fun getUrl(state: String, token: Token): String {
+        if (profile == "live") {


프로필을 한번 Wraaping하는 객체가 있으면 어떨까 싶은데, 한번 같이 고민해주실 수 있나요?
profile 값을 갖는 유틸이 있어서, 그 유틸 객체에게 메서드로 질의하는 방법도 괜찮을 것 같아서요

ProfileHolder or ProfileUtil이라는 이름의 객체가 있다면
ProfileHolder.isLive(), ProfileHolder.isDev() 와 같은 식으로 메서드를 통해 질의하면,
제가 자주 경계하는 실수의 가능성이 있는 raw한 string의 사용을 없앨 수도 있고, 객체지향 적으로도 좋은 것 같고, 코드도 깔끔해질 수도 있을 것 같아요. 그리고 리팩토링시 인텔리제이의 도움을 받기도 좋을 것 같아요.

더 나아가 괜찮은 구현인지는 모르겠지만, 상태에 따라 RedirectURL을 반환하는 객체가 있는 것도 괜찮을 것 같아요.
ProfileRedirectUrlHolder.getRedirectUrl()과 같이 스스로 Profile을 확인하고, 그에 맞는 Redirect URL을 반환하는 건 어떨까 해서요. ex) live의 경우 "https://bandiboodi.com 반환

토큰 값에 대한 파라미터는 따로 붙여주거나, 따로 붙여주는 Util 클래스가 있거나, 아니면 토큰 값들을 넣으면 토큰 값까지 모두 포함된 RedirectUrl을 반환하는 객체가 있는 것도 괜찮을 것 같아요 (마지막은 과한 것 같기도 합니다)

객체를 어떤 단위까지 분리할까는 항상 고민이 되는 부분인 것 같습니다 ㅎㅎ
진호님이 말씀하신 걸 정리해보면 아래와 같이 객체가 생겨날 것 같습니다

ProfileHolder(or ProfileUtil) -> profile 값을 주입 받아 현재 application의 profile을 반환하는 책임

ProfileRedirectUrlHolder -> referer값을 기반으로 RedirectUrl의 host + path까지 반환 책임

토큰 값을 붙여주는 역할의 객체 -> token을 받아 query string으로 붙여주는 책임

최종 redirectUrl을 반환하는 객체

사실 저는 작업하면서 이렇게까지 객체를 분리하지는 않았던 것 같습니다.
하지만 진호님이 제시해주신 ProfileHolder.isLive(), ProfileHolder.isDev() 메서드가 너무 깔끔해보인다는 생각이 들었고,
StateOAuth2AuthorizationRequestRepository에서 재사용된다는 장점을 생각했을 때 적용해보고 싶다는 생각이 들었습니다 ㅎㅎ

우선은 ProfileHolder로 현재 application의 profile을 알려주는 책임을 가지는 객체 만들어 반영해보겠습니다.
정성스러운 리뷰 감사합니다 ❤️

binary-ho · 2024-03-27T16:45:43Z

backend/application/api/src/main/kotlin/io/raemian/api/support/security/LoginRedirector.kt

+
+@Component
+class LoginRedirector(
+    @Value("\${spring.profiles.active:local}")


최근에 이런 방법도 봤네요. 값들이 모여있고, 객체로 관리하고, 리팩토링 하기 쉬운 장점이 있는거 같은데 한번 슥 봐주세요 절대 이거 도입해보자 아닙니다!!
https://medium.com/@mikael_55667/why-you-should-stop-using-value-annotations-in-spring-and-use-this-instead-2c8a47e5096a

저는 진호님이 얘기해주신 ConfigurationProperties 사용하는 방식 좋아합니다!
저희 프로젝트에 AppleLoginProps 객체에 사용하기도 했구요
다만 저는 한번에 많은 값을 주입해야 할 때 ConfigurationProperties를 고려했던 것 같습니다!
Profile을 담을 객체에 주입할 값이 많아진다면 ConfigurationProperties 사용하는 방식 좋은 것 같아요 👍

binary-ho · 2024-03-27T16:50:29Z

backend/application/api/src/main/kotlin/io/raemian/api/support/security/LoginRedirector.kt

+
+        val referer = loginRequestRefererStorage.get(state)
+
+        if (profile == "dev" && referer.contains("dev-bandiboodi")) {


저번에 말씀 드렸던 대로 dev 클라이언트 유출에 대한 우려가 있습니다.
물론 저도 실제로 사용하는 사람이 있긴 할까..싶기는 한데, 저희가 관리하지 않는 데이터가 dev-db에 저장될 수도 있는 상황이 조금 꺼려집니다. 어떻게 생각하실까요?

앗!! 이번에 제거한다는 걸 깜빡했네요 ㅎㅎ
확인 감사해요 👍

별거 아닌데 프로필에 대한 비교값을 문자열이 아닌 enum 으로 관리하는게 좋긴할듯 (referer 도 enum 으로)

아 위에 진호가 말했구나 ㅎㅎ

같은 부분을 형은 enum으로, 진호는 현재 profile을 원하는 값과 비교해주는 객체 메서드로 바꾸는 방향을 제안해줬는데
둘 다 적용해보겠습니다 👍

binary-ho · 2024-03-27T17:02:13Z

...pplication/api/src/main/kotlin/io/raemian/api/support/security/LoginRequestRefererStorage.kt

+        .expireAfterWrite(60L, TimeUnit.SECONDS)
+        .build<String, String>()
+
+    fun put(state: String, referer: String?) {


[[ 100% 취향 제안 ]]
저도 자바에 익숙해서 우석님이랑 항상 같은 방식으로 코드를 작성했는데요?
요즘 코틀린 코드들 다른데서 보면서 아래와 같은 코드도 조금 이뻐 보이이기 시작했는데 어떠신가요..

put() = if (referer.isNullOrBlank()) return else timedStorage.put(state, referer)

혹은.. 확장함수 활용해서 isNotNullOrBlank() 구현한 다음..

fun put() { if (referer.isNotNullOrBlank()) timedStorage.put(state, referer) }

완전 취향이고 지금 얼리 리턴 방식도 너무 좋아요 저도 보통 그렇게 작성해왔던 것 같아요.
어떤지 한번 얘기해보고 싶어서 올려봐요!

저는 얼리 리턴을 위한 조건문은 명시적으로 작성하는 편이 잘 읽힌다고 생각하는 편이였는데요.
진호님이 제시해주신 첫번째 코드 (if문 Expression으로 사용)를 보니 한번 사용해보고 싶다는 생각이 들었습니다 ㅎㅎ
첫번째 방식으로 수정 한번 해보겠습니다!
제안 감사합니다 👍

@binary-ho
제안 감사합니다 👍

추가 제안 ㅆㄱㄴ

binary-ho · 2024-03-27T17:08:50Z

...src/main/kotlin/io/raemian/api/support/security/StateOAuth2AuthorizationRequestRepository.kt

@@ -25,6 +30,11 @@ class StateOAuth2AuthorizationRequestRepository() : AuthorizationRequestReposito
        response: HttpServletResponse,
    ) {
        if (authorizationRequest != null) {
+            /*** for frontend dev test ***/
+            if (profile == "dev") {
+                loginRequestRefererStorage.put(authorizationRequest.state, request.getParameter("referer"))


프로필래핑 객체가 있다면 여기서도 활용이 가능하다는 장점 ㅎㅎ (어필)

제안 감사합니다 👍

추가 제안 ㅆㄱㄴ

egg528 · 2024-03-28T14:35:53Z

backend/application/api/src/main/kotlin/io/raemian/api/support/security/ProfileType.kt

@@ -0,0 +1,17 @@
+package io.raemian.api.support.security
+
+enum class ProfileType(


@ManHyuk
제안 감사합니다 👍

추가 제안 ㅆㄱㄴ

egg528 · 2024-03-28T14:39:26Z

backend/application/api/src/main/kotlin/io/raemian/api/support/security/RedirectUrlHolder.kt

+
+import org.springframework.boot.context.properties.ConfigurationProperties
+
+@ConfigurationProperties(prefix = "redirect-url")


@binary-ho
2개의 제안을 합쳐서 RedirectUrlHolder에 ConfigurationProperties를 적용해봤습니다!
제안 감사합니다 👍

추가 제안 ㅆㄱㄴ

egg528 · 2024-03-28T14:40:22Z

backend/application/api/src/main/kotlin/io/raemian/api/support/security/ProfileHolder.kt

+import org.springframework.stereotype.Component
+
+@Component
+class ProfileHolder(


@binary-ho
제안 감사합니다 👍

추가 제안 ㅆㄱㄴ

binary-ho

LGTM!

댓글로 변경된 부분도 짚어주는 센스 덕분에 무엇을 어떻게 바꾸었는지 빠르게 파악할 수 있었습니다! 작업 너무 고생 많으셨습니다 감사합니다 ㅎㅎㅎ

@FunctionalInterface

* feat: remove forward header optiion * refactor : birth 관련 유저 정보 업데이트 API에 birth를 nullable로 변경 (#206) (#207) * feat(#209): spring security authorization_request_not_found 수정 (#210) * [🌎 Feature] Goal 댓글 API 개발 (#208) * feat : 댓글 엔티티와 repository 생성 (#196) * feat : Goal 댓글 오름차순 전체 조회 메서드 구현 (#196) * feat : Goal 댓글 작성 삭제 메서드 구현 (#196) * feat : Goal의 마지막 댓글 Read 시간을 저장하는 GoalCommentReadTime 구현 (#196) * feat : Goal의 생성자가 읽지 않은 댓글이 있는지 확인하는 메서드 구현 (#196) * feat : 비동기적으로 마지막 댓글 확인 시간을 update하는 메서드 구현 (#196) * ktlint formatting * refactor : 마지막 댓글 Read 시간을 분리된 테이블이 아닌 Goal이 갖도록 변경 (#196) * feat : 댓글 조회시 요청자가 Goal의 주인이라면 comment read time을 update하는 이벤트 발행 (#196) * test : Goal 필드 추가에 따른 테스트 코드 변경 (#196) * refactor : CoreApiException이 직접 errorMessage를 입력 받을 수 있도록 변경 (#196) * refactor : Comment가 스스로 제한 조건을 검증하도록 변경 (#196) * refactor : CommentService의 필드들에 private 접근 제한자 추가 (#196) * refactor : comment require 조건 정정 (#196) * refactor : 불필요한 트랜잭션 전파 및 Transactional 이벤트 리스너 제거 (#196) * refactor : 불필요한 Goal 조회 쿼리 제거 (#196) * test : Comment 관련 기능 통합 테스트 코드 작성 (#196) * ktlint formatting * refactor : getAllByGoalId 메서드의 이름을 역할에 맞게 findAllByGoalId로 변경 (#196) * feat : 댓글 API 구현 (#196) * chore : 잘못 작성된 메서드 이름 변경 (#196) * refactor : DTO 패키지 변경 (#196) * refactor : comments에서 goal을 꺼낼 때, first 메서드 활용 (#196) * refactor : 불필요한 @async 어노테이션 제거 (#196) * refactor : findAllByGoalId 가독성 개선 (#196) * refactor : Emoji API Restful하게 개선 (#196) (#215) * [🚌 Issue ] Emoji Count 작업 및 Count 관련 로직 개선 (#213) * refactor(#211): count관련 eventListener 단일 component화 * feat(#211): emoji count 테이블 추가 및 count관련 event application lock 적용 * feat(#211): add goal count event에 exclusive runner 적용 * refactor: event 같은 패키지로 이동 * feat: lock 적용 시점 비동기 이벤트 실행 Thread로 변경 * refactor: event 네이밍 컨벤션 적용 * fix: key에 매핑된 lock 대기 thread가 없다면 map에서 제거 * [🚌 Issue] 리팩토링 이슈 템플릿 세팅 (#220) * chore : 리팩토링 이슈 템플릿 세팅 * chore : 기타-이슈 기본 생성 제목에 불필요한 띄어쓰기 제거 * [🛠️ Refactor] Goal Emoji API 응답에 "이모지를 누른 사람 수" 추가 (#221) * refactor : Goal에 반응 조회 API에 반응한 사람 수 값 추가 (#218) * test : Goal 반응 조회 API 반응한 사람 수 값이 정확한지 확인하는 테스트 작성 (#218) * [🛠️ Refactor] IsMyReaction을 확인할 때 Username이 아닌 id로 확인하기 (#219) * refactor : IsMyReaction을 확인할 때 Username이 아닌 id로 확인하도록 변경 (#217) * merge with develop * [🆘 Bug] authorization_request_not_found 오류 (#216) * feat(#209): spring security authorization_request_not_found 수정 * fix: AuthorizationRequestRepository cookie 기반에서 state parameter 기반으로 변경 * fix: AuthorizationRequestRepository remove 메서드에 명시적인 삭제 추가 * fix: AuthorizationRequestRepository oauthRequestStorage 데이터 명시적 삭제 로직 제거 * refact(#171): add emoji count * feat(#171): 피드에 이모지및 내가 누른 이모지 추가 * chore(#171): ktlint * chore(#171): 사용하지 않는 코드 삭제 * chore(#171): ktlint * [🛠️ Refactor] 댓글 응답에 댓글 갯수 추가 (#225) * refactor : Goal에 달린 댓글 응답에 댓글 갯수 추가 (#224) * test : Goal에 달린 댓글 응답이 댓글 갯수를 반환하는 것을 확인하는 테스트 코드 작성 (#224) * ktlint formatting * refact(#171): edit field name * [🛠️ Refactor] RunnerLock 필드 변수명 변경 �+ Security Uri 상수화 + H2 웹 콘솔용 WebSecurityCustomizer 제거 (#226) * refactor: RunnerLock 내부 필드 변수명 변경 * refactor: security permitall uri 상수화 * [🛠️ Refactor] 댓글 조회 API 응답에 "내 지도 확인" key-value 추가 (#230) * refactor : CommentsResponse에 자신의 목표인지 확인하는 isMyGoal 필드 추가 (#229) * test : CommentsResponse를 통해 자신의 목표인지 확인할 수 있는지 검증하는 테스트 코드 작성 (#229) * feat(#232): add title field * [🛠️ Refactor] 프로젝트 구조 변경 (1번, 3번 제안만 변경) (#234) * refactor(#235): pagination 공통화 + cheering squad 공통 pagination 적용 * [🛠️ Refactor] 잘못 작성된 읽지 않은 댓글 조회 쿼리 올바르게 정정 (#237) * refactor : CommentsResponse에 자신의 목표인지 확인하는 isMyGoal 필드 추가 (#229) * test : CommentsResponse를 통해 자신의 목표인지 확인할 수 있는지 검증하는 테스트 코드 작성 (#229) * refactor : exist new comment 쿼리를 id와 시간으로 검색하도록 변경 (#236) * refactor : 쿼리가 Goal Id를 사용하도록 변경 (#236) * test : 본인의 Goal만 조회하는지 검증하는 테스트 추가 작성 (#236) * feat: pagination 공통 response 추가 + cheering squad 조회를 위한 queryRepository 추가 * feat(#235): comment count 엔티티 추가 + comment create/delete event 생성 * feat(#235): 목표 타임라인 페이징 조회 API 작업 * fix: dev 환경 로그인 callback 시 redirect uri 변경 * chore: apply lint * refactor: CheererRepository 미사용 메서드 제거 * refactor: TriFunction @FunctionalInterface 적용 * refactor(#235): PR 코멘트 반영 * [🌎 Feature] 타임 라인용 목표 조회 API 개발 (#240) * refactor(#235): pagination 공통화 + cheering squad 공통 pagination 적용 * feat: pagination 공통 response 추가 + cheering squad 조회를 위한 queryRepository 추가 * feat(#235): comment count 엔티티 추가 + comment create/delete event 생성 * feat(#235): 목표 타임라인 페이징 조회 API 작업 * fix: dev 환경 로그인 callback 시 redirect uri 변경 * chore: apply lint * refactor: CheererRepository 미사용 메서드 제거 * refactor: TriFunction @FunctionalInterface 적용 * refactor(#235): PR 코멘트 반영 * hotfix(#235): 타인 타임라인 조회 API 추가 * chore: apply klint * [🛠️ Refactor] 타임 라인 API 작업 리팩토링 (#241) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * hotfix: dev login redirect url 수정 * hotfix: login redirect url referer 기준 분기 처리 * hotfix: login redirect url referer 기준 분기 처리 조건 추가 * hotfix: login redirect url referer 기준 분기 처리 제거 * [🌎 Feature] 타임 라인용 목표 조회 API 개발 - Cursor Deadline으로 변경 (#242) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * fix: 목표 피드 조회 API deadline 역순으로 정렬 조회 * chore: apply lint * fix: goals 테이블의 index 수정 * refactor: pagination cusorId type Any -> Generic으로 변경 * [🌎 Feature] 타임 라인용 목표 조회 API 개발 - Pagination 방식 offset 기반으로 변경 (#243) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * fix: 목표 피드 조회 API deadline 역순으로 정렬 조회 * chore: apply lint * fix: goals 테이블의 index 수정 * refactor: pagination cusorId type Any -> Generic으로 변경 * fix: 타임 라인 목표 조회 API paging 처리 방식 offset으로 변경 * refactor: cursor 기반 pagination 공통 모듈 cursor type long으로 통일 * refactor: pagination 공통 모듈 cursor와 offset 구분 * refactor: 미사용 코드 및 의미 없는 중복 코드 삭제 * hotfix: global exception config 내 NoSuchElementException 설정 수정 * [🚌 Issue] dev api 로그인 요청 위치에 따라 redirect url 변환 (#245) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * fix: 목표 피드 조회 API deadline 역순으로 정렬 조회 * chore: apply lint * fix: goals 테이블의 index 수정 * refactor: pagination cusorId type Any -> Generic으로 변경 * fix: 타임 라인 목표 조회 API paging 처리 방식 offset으로 변경 * refactor: cursor 기반 pagination 공통 모듈 cursor type long으로 통일 * refactor: pagination 공통 모듈 cursor와 offset 구분 * feat(#244): dev api 로그인 요청 위치에 따라 redirect url 변환 * chore: apply lint * fix: @value(spring.profiles.active) default값 추가 * refactor: 코드 리뷰 반영(#245) * chore: encrypt하던 데이터 삭제 * chore: encrypted test properties 삭제 * chore: referer 저장/조회 logging 추가 * chore: logging 제거 * fix: getReferer 메서드 referer값 null일 경우 Empty String 반환 * [🆘 Bug] 타임라인 관련 버그 수정 (#248) * fix: timeline 조회 시 목표 없으면 early return * feat: 유저정보 기반 타임 라인 조회 API 작업 * fix: 목표 삭제 시 LifeMapCount goalCount 필드 -1 처리 * refactor: timeline 조회 api 내 목표 개수 조회 쿼리 변경 * feat(#171): edit comment count * chore: ktlint * refactor : Comment와 Emoji에 Goal Delete Casacade 속성 추가 (#246) (#249) * refactor : 자신의 Goal에 달린 Comment 삭제를 허용하도록 변경 (#251) (#252) * bug(#254): 타임라인 조회 API 내 코멘트 개수 조회 쿼리 오류 수정 * feat : 이모지 추가 전에 같은 이모지가 존재하는지 확인하는 메서드 작성 (#253) (#256) * refactor : 모든 Emoji 조회 API에서 일관되게 Emoji id 순서로 정렬해 응답하도록 변경 (#257) * refactor : 불필요한 정렬 제거 - MySQL Clustered Index로 인해 이미 pk 정렬 되어 있다 (#257) * fix : 피드 조회 (#259) * feat(#171): 목표 생성시 기본 댓글 개수 추가 * chore(#171): ktlint * feat(#171): 기본 카운트 추가 * chore: ktlint * refact(#171): 변수 삭제 --------- Co-authored-by: ManHyuk <[email protected]> Co-authored-by: binary_ho <[email protected]>

egg528 added 18 commits March 24, 2024 11:34

delete: 유저 정보를 활용한 타임라인 조회 API 삭제

b99367c

refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일

4249f81

refactor: 메서드 로직에 따라 get, find 명칭 구분

854f4a2

refactor: lamda body 내 생략 가능한 parameter 생략

23aa5f9

refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCount…

1c7d46f

…Map)

refactor: associateWith 메서드 활용

74b372b

refactor: PaginationResult total값 Long으로 다시 변경

54c8b81

Merge branch 'develop' of github.com:depromeet/amazing3-be into feature/

ef9f69d

#235

fix: 목표 피드 조회 API deadline 역순으로 정렬 조회

30b557b

Merge branch 'develop' of github.com:depromeet/amazing3-be into feature/

89264c7

#235

chore: apply lint

5425558

fix: goals 테이블의 index 수정

48e2709

refactor: pagination cusorId type Any -> Generic으로 변경

a9135a1

Merge branch 'develop' of github.com:depromeet/amazing3-be into feature/

63bb7e9

#235

fix: 타임 라인 목표 조회 API paging 처리 방식 offset으로 변경

4c110bc

refactor: cursor 기반 pagination 공통 모듈 cursor type long으로 통일

bb151c4

refactor: pagination 공통 모듈 cursor와 offset 구분

0a00a11

feat(#244): dev api 로그인 요청 위치에 따라 redirect url 변환

c94c5b5

egg528 self-assigned this Mar 27, 2024

egg528 requested review from ManHyuk and binary-ho as code owners March 27, 2024 14:54

chore: apply lint

e97bc4e

egg528 changed the base branch from develop to feature/#235 March 27, 2024 15:17

fix: @value(spring.profiles.active) default값 추가

583201f

binary-ho reviewed Mar 27, 2024

View reviewed changes

Base automatically changed from feature/#235 to develop March 28, 2024 13:08

refactor: 코드 리뷰 반영(#245)

e307c2d

egg528 commented Mar 28, 2024

View reviewed changes

egg528 added 3 commits March 28, 2024 23:42

chore: encrypt하던 데이터 삭제

1073e18

chore: resolve conflict

39ce4cc

chore: encrypted test properties 삭제

128357d

ManHyuk approved these changes Mar 29, 2024

View reviewed changes

binary-ho approved these changes Mar 31, 2024

View reviewed changes

chore: referer 저장/조회 logging 추가

e27bb5b

egg528 merged commit 4f570c2 into develop Mar 31, 2024
1 check passed

egg528 deleted the feature/#244 branch March 31, 2024 11:22

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[🚌 Issue] dev api 로그인 요청 위치에 따라 redirect url 변환 #245

[🚌 Issue] dev api 로그인 요청 위치에 따라 redirect url 변환 #245

egg528 commented Mar 27, 2024 •

edited

Loading

binary-ho commented Mar 27, 2024

egg528 commented Mar 27, 2024

binary-ho Mar 27, 2024

egg528 Mar 28, 2024 •

edited

Loading

binary-ho Mar 27, 2024

egg528 Mar 28, 2024 •

edited

Loading

binary-ho Mar 27, 2024

egg528 Mar 28, 2024

ManHyuk Mar 28, 2024

ManHyuk Mar 28, 2024

egg528 Mar 28, 2024 •

edited

Loading

binary-ho Mar 27, 2024

egg528 Mar 28, 2024

egg528 Mar 28, 2024

binary-ho Mar 27, 2024

egg528 Mar 28, 2024

egg528 Mar 28, 2024 •

edited

Loading

egg528 Mar 28, 2024

egg528 Mar 28, 2024

binary-ho left a comment •

edited

Loading


		val referer = loginRequestRefererStorage.get(state)

		if (profile == "dev" && referer.contains("dev-bandiboodi")) {

		@@ -0,0 +1,17 @@
		package io.raemian.api.support.security

		enum class ProfileType(


		import org.springframework.boot.context.properties.ConfigurationProperties

		@ConfigurationProperties(prefix = "redirect-url")

[🚌 Issue] dev api 로그인 요청 위치에 따라 redirect url 변환 #245

[🚌 Issue] dev api 로그인 요청 위치에 따라 redirect url 변환 #245

Conversation

egg528 commented Mar 27, 2024 • edited Loading

📒 Issue

🎯 어떤 작업을 했는지

🌍 영향범위 (모듈)

binary-ho commented Mar 27, 2024

egg528 commented Mar 27, 2024

Choose a reason for hiding this comment

egg528 Mar 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

egg528 Mar 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

egg528 Mar 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

egg528 Mar 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

binary-ho left a comment • edited Loading

Choose a reason for hiding this comment

egg528 commented Mar 27, 2024 •

edited

Loading

egg528 Mar 28, 2024 •

edited

Loading

egg528 Mar 28, 2024 •

edited

Loading

egg528 Mar 28, 2024 •

edited

Loading

egg528 Mar 28, 2024 •

edited

Loading

binary-ho left a comment •

edited

Loading