Fix/#128 change http library #133

yummygyudon · 2023-10-16T06:05:00Z

📝 PR Summary

기존 RestTemplate을 OpenFeign으로 마이그레이션

코드 간결성 개선 목적
코드 가독성 개선 목적
관리 용이성 개선

🌵 Working Branch

fix/#128-change-http-library

🌴 Works

[ ]
[ ]

🌱 Related Issue

#128

- 유기적인 Header 생성 가능 - 단, springframework.web.bind 패키지의 Mapping Annotation 쓰기 불편함 - Method 인식 에러 발생 -> spring.cloud.openfeign 을 사용하지 않은 이유 - 어노테이션 최소화

gunom

좋은 것 같습니다! 다만 하나 고민이 생긴 것이 "client가 과연 application layer에 적합한가?" 가 있네요. 유사 MSA를 사용 중이라 client의 역할이 외부 서버에서 응답을 받아오는 거라면 application보다 interface에 더 적합하다는 생각도 들어요! 한번 고민해보면 좋을 거 같아요!

gunom · 2023-10-26T08:03:15Z

src/main/java/org/sopt/app/common/config/ClientConfig.java

+    public OkHttpClient okHttpClient() {
+        return new OkHttpClient();
+    }
+//    @Bean


주석 지우기!

gunom · 2023-10-26T08:05:41Z

src/main/java/org/sopt/app/application/operation/OperationService.java

-
-        val entity = new HttpEntity(null, headers);
-
+        Map<String, String> headers = Map.of(


뭔가 header 만드는게 공통적인게 있을 거 같은데 따로 뺄 방법이 있으면 좋을 것 같다는 생각이 드네요!
다른 서비스 클래스에서도 content-type과 Authorization을 공통적으로 사용하는 애들은 모두 해당 코드가 반복될 수 있겠다는 생각...?

아합!! 요부분 util static 함수나 내부 private 함수로 분리 해보도록 하겠습니다!

@gunom 요부분 더 고민해봤는데
playground token refresh api는 accessToken 이 없고
x-api-key & x-request-from 이라는 특수 헤더 값이 필요로 하더라구요!!

그래서 헤더는 Service마다 특수성이 있을 것 같기도 하나
메서드로 분리하는게 가독성 개선에 도움이 될 것 같다는 부부엔 전적으로 동의하여
우선 각 Service 클래스에서 사용하는 공통 헤더 값을 포함하는 Header Map 객체 생성 메서드를 private으로 선언하여 재사용하도록 했습니다!!

이후 커밋 봐주시면 될 것 같아옹~~

yummygyudon · 2023-10-26T08:23:15Z

좋은 것 같습니다! 다만 하나 고민이 생긴 것이 "client가 과연 application layer에 적합한가?" 가 있네요. 유사 MSA를 사용 중이라 client의 역할이 외부 서버에서 응답을 받아오는 거라면 application보다 interface에 더 적합하다는 생각도 들어요! 한번 고민해보면 좋을 거 같아요!

오오,....app 으로 기준을 잡고 생각하다보니 service 내부에서 데이터를 가져오는 용도로 client가 사용되기에 application layer에 포함기켜 놓았는데
건모씨의 의견도 매우 좋은 것 같습니다..!!
흠... 말씀하신 방법대로라면 interface 패키지 내부에 external 폴더를 만들어서 client 들을 관리하는 건 어떨까요?!

- 외부 서버에서 응답을 받아오는 객체들로서 application 계층보다는 interface 계층에 있는 것이 바람직하다고 판단

yummygyudon added 2 commits October 16, 2023 14:49

FEAT : OpenFeign 의존성 추가 및 Application 에 세팅

238e69a

FEAT : PlaygroundClient 생성

9c24da8

yummygyudon self-assigned this Oct 16, 2023

yummygyudon linked an issue Oct 16, 2023 that may be closed by this pull request

[FIX] RestTemplate to OpenFeign #128

Closed

1 task

pull-request-size bot added the size/S label Oct 16, 2023

yummygyudon added 6 commits October 19, 2023 04:43

FIX : Spring Cloud OpenFeign 해제 및 GitHub OpenFeign 설정 추가

5eb3f6c

REFACTOR : RestTemplate migrate to OpenFeign - PlayGround

b526886

REFACTOR RestTemplate Migrate to OpenFeign

8814fc1

CHORE : FeignClient Config

4022358

FIX : PushToken RestTemplate 사용 유지 및 Delete 메서드 파라미터 수정

96eac52

pull-request-size bot added size/L and removed size/S labels Oct 21, 2023

yummygyudon requested a review from gunom October 23, 2023 17:26

gunom reviewed Oct 26, 2023

View reviewed changes

yummygyudon added 4 commits October 27, 2023 23:16

FIX : 불필요한 주석 및 필드 지우기

8eb0100

REFACTOR : 패키징 개선

d441d01

- 외부 서버에서 응답을 받아오는 객체들로서 application 계층보다는 interface 계층에 있는 것이 바람직하다고 판단

REFACTOR : Header Map 생성 메서드 분리

2b834e2

Merge branch 'dev' into fix/#128-change-http-library

1ceeb4c

yummygyudon merged commit dc90b7b into dev Nov 16, 2023
0 of 2 checks passed

ozzing deleted the fix/#128-change-http-library branch May 23, 2024 16:01

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix/#128 change http library #133

Fix/#128 change http library #133

yummygyudon commented Oct 16, 2023

gunom left a comment

gunom Oct 26, 2023

gunom Oct 26, 2023

yummygyudon Oct 26, 2023

yummygyudon Nov 13, 2023

yummygyudon commented Oct 26, 2023


		val entity = new HttpEntity(null, headers);

		Map<String, String> headers = Map.of(

Fix/#128 change http library #133

Fix/#128 change http library #133

Conversation

yummygyudon commented Oct 16, 2023

📝 PR Summary

🌵 Working Branch

🌴 Works

🌱 Related Issue

gunom left a comment

Choose a reason for hiding this comment

gunom Oct 26, 2023

Choose a reason for hiding this comment

gunom Oct 26, 2023

Choose a reason for hiding this comment

yummygyudon Oct 26, 2023

Choose a reason for hiding this comment

yummygyudon Nov 13, 2023

Choose a reason for hiding this comment

yummygyudon commented Oct 26, 2023