✨ use oracle object storage to upload image #426
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
HaiSeong
reviewed
Nov 28, 2024
Comment on lines
-2
to
+6
| id 'java' | ||
| id 'org.springframework.boot' version '3.2.7' | ||
| id 'io.spring.dependency-management' version '1.1.5' | ||
| id 'jacoco' | ||
| id 'com.epages.restdocs-api-spec' version '0.18.2' | ||
| id 'java' | ||
| id 'org.springframework.boot' version '3.2.7' | ||
| id 'io.spring.dependency-management' version '1.1.5' | ||
| id 'jacoco' | ||
| id 'com.epages.restdocs-api-spec' version '0.18.2' |
There was a problem hiding this comment.
기존은 tab 기반으로 gradle 작성했었고 변경된건 스페이스4개 기준이네요.
혹시 어떤게 컨벤션에 맞는건가요?
There was a problem hiding this comment.
gradle indent 컨벤션에 대해서 저희끼리 논의해봐야 할 것 같아요😅
| private final S3ClientService s3ClientService; | ||
| private final ImageClientService imageClientService; |
Comment on lines
43
to
45
| } catch (Exception e) { | ||
| throw new RuntimeException("Failed to load private key", e); | ||
| } |
There was a problem hiding this comment.
예외가 터졌을때 Oracle Object Storage 문제라는것을 바로 알 수 있게 메시지에 추가하는것도 좋아보입니다!
물론 에외에 이미 나와있을것 같아서 선택적으로 반영하면 좋을것 같아요!
There was a problem hiding this comment.
추가적으로 예외 메시지가 한글인 게 좋을 것 같아 이 코멘트를 반영하며 함께 수정했습니다!
| @Value("${oracle.cloud.url-prefix}") | ||
| private String urlPrefix; | ||
|
|
||
| private static final int DURATION_SECONDS = 600; |
There was a problem hiding this comment.
이 시간은 어떤값인가요? 600인 기준은 어떤 이유인가요?
또는 이 값을 yaml로 이동시키는것도 고려 해보셨나요!?
There was a problem hiding this comment.
이미지 업로드를 위해 제공하는 PAR의 만료 시간입니다.
펭쿡이 이전에 사용하던 presigned url에 부여하던 만료기한 규칙과 동일하게 적용했습니다.
yaml로 해당 값을 관리하는게 유지보수에 더 용이하겠네요! 감사합니다
Comment on lines
+56
to
+61
| return new ImageUrlResponse(String.format("%s/n/%s/b/%s/o/%s", | ||
| client.getEndpoint(), | ||
| namespace, | ||
| bucketName, | ||
| fileName)); | ||
| } |
There was a problem hiding this comment.
오라클에서 제공하는 url 형식입니다
리소스 식별하는 구분자로 사용됩니다!
/n namespace
/b buckeName
/o object
HaiSeong
approved these changes
Nov 28, 2024
|
|
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
S3 를 사용하던 이미지 업로드/조회를 Object Storage를 사용하도록 변경했습니다.
기본적인 로직은 기존 로직과 동일합니다.
S3에서 presigned url이었던 업로드용 url을 Object Storage의 PAR로 변경했고,
변경이 용이하게 하기 위해 인터페이스로 추상화했습니다.
어떤 구현체를 사용할지는 ImageConfig에서 주입해서 정할 수 있습니다.
UserService, RecipeService, RecipeStepService는 이제 우리가 S3를 쓰는지 Object Storage를 쓰는지 모릅니다!!!!!!
(멋지죠...)
변경사항은 다음과 같습니다.
기존에는 AWS S3에 요청을 전송하는 서버 자체로 인증을 했으나, 현재는 API key를 발급받아 이것을 직접 사용합니다.
참고자료
오라클 공식문서
Maven 버전