fix: 이미지 url을 이미지 확장자로 끝나도록 함 #137
Open
Conversation
This file contains hidden or 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
Walkthrough이미지 파일 확장자가 없는 경우를 처리하기 위해 새로운 에러 코드가 추가되었고, 이미지 분석 요청 시 URL에서 확장자를 감지하여 잘라내는 로직이 도입되었습니다. 또한, Swagger 문서에 확장자 누락 시 발생하는 400 에러 응답이 명시적으로 추가되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant Controller
participant OpenAiClient
participant ErrorHandler
Client->>Controller: 이미지 분석 요청 (image URL 포함)
Controller->>OpenAiClient: explainImage(imageUrl)
OpenAiClient->>OpenAiClient: trimImageUrl(imageUrl)
alt 확장자 존재
OpenAiClient->>OpenAiClient: URL 확장자 기준으로 자름
OpenAiClient->>Controller: 분석 결과 반환
Controller->>Client: 200 OK + 결과
else 확장자 없음
OpenAiClient->>ErrorHandler: BusinessException(NO_IMAGE_FILE_EXTENSION_FOUND)
ErrorHandler->>Client: 400 Bad Request + 에러 메시지
end
Suggested reviewers
Poem
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🔭 Outside diff range comments (1)
src/main/java/com/webeye/backend/imageanalysis/infrastructure/OpenAiClient.java (1)
192-206: 💡 Verification agent🧩 Analysis chain
다른 이미지 분석 메서드들에서도 URL 검증을 고려해보세요.
현재
trimImageUrl은explainImage에서만 사용되고 있습니다. 다른 이미지 분석 메서드들(explainAllergy,explainNutrition,explainCosmetic등)에서도 동일한 URL 검증이 필요할 수 있습니다.다른 메서드들에서 URL 검증 필요성을 확인해보겠습니다:
만약 다른 메서드들에서도 URL 검증이 필요하다면,
callWithStructuredOutput메서드 내부에서 URL들을 전처리하는 것이 더 일관성 있는 접근법일 수 있습니다.
🏁 Script executed:
#!/bin/bash # 이미지 분석 관련 메서드들에서 URL 사용 패턴 확인 rg -A 10 -B 5 "callWithStructuredOutput.*urls" src/main/java/com/webeye/backend/imageanalysis/infrastructure/OpenAiClient.javaLength of output: 5134
URL 전처리 로직을 모든 메서드에 일관되게 적용하세요
현재
trimImageUrl은explainImage에서만 사용되고 있어, 외부에서 전달된 URL 리스트(List<String> urls)를 처리하지 않고 있습니다. 모든 이미지 분석 메서드에서 동일한 전처리를 적용하거나,callWithStructuredOutput내부로 중앙화하여 한 번만 처리하도록 변경하는 것이 좋습니다.• 대상 파일:
src/main/java/com/webeye/backend/imageanalysis/infrastructure/OpenAiClient.java
• 메서드:explainAllergy,explainNutrition,explainCosmetic,explainHealthFood
• 호출부:return callWithStructuredOutput(urls, …);• 개선안 (예시)
private <T> T callWithStructuredOutput(List<String> urls, ImageAnalysisPrompt prompt, Class<T> clazz) { - BeanOutputConverter<T> outputConverter = new BeanOutputConverter<>(clazz); + // URL 전처리: 공백·특수문자 제거 등 + List<String> trimmedUrls = urls.stream() + .map(this::trimImageUrl) + .toList(); + BeanOutputConverter<T> outputConverter = new BeanOutputConverter<>(clazz); - for (String imageUrl : urls) { + for (String imageUrl : trimmedUrls) { MimeType extension = ImageMimeType.fromExtension(extractFileExtension(imageUrl)); promptUserSpec.media(extension, new UrlResource(imageUrl)); }위와 같이 중앙화하면 추후 메서드 추가 시에도 URL 검증 코드를 반복 작성할 필요가 없어집니다.
🧹 Nitpick comments (1)
src/main/java/com/webeye/backend/imageanalysis/presentation/swagger/ImageAnalysisSwagger.java (1)
23-26: API 에러 응답 문서화가 적절히 추가되었습니다.새로운 에러 케이스에 대한 명시적 문서화로 API 사용자에게 도움이 됩니다. 에러 메시지도 ErrorCode와 일치하여 일관성이 있습니다.
다른 가능한 에러 상황들도 문서화를 고려해보세요:
- 잘못된 이미지 URL (400)
- OpenAI 서버 에러 (500)
- 지원하지 않는 이미지 형식 (400)
@ApiResponse( responseCode = "400", description = "이미지 확장자가 없습니다." + ), + @ApiResponse( + responseCode = "400", + description = "잘못된 이미지 URL이거나 지원하지 않는 이미지 형식입니다." + ), + @ApiResponse( + responseCode = "500", + description = "OpenAI 서버 에러가 발생했습니다." )
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
src/main/java/com/webeye/backend/global/error/ErrorCode.java(1 hunks)src/main/java/com/webeye/backend/imageanalysis/infrastructure/OpenAiClient.java(3 hunks)src/main/java/com/webeye/backend/imageanalysis/presentation/swagger/ImageAnalysisSwagger.java(1 hunks)
🔇 Additional comments (3)
src/main/java/com/webeye/backend/global/error/ErrorCode.java (1)
16-16: 새로운 이미지 확장자 에러 코드가 적절히 추가되었습니다.기존의
FILE_EXTENSION_NOT_FOUND와 구별되는 용도로 잘 정의되었습니다. 에러 메시지도 명확하고 HTTP 상태 코드도 적절합니다.src/main/java/com/webeye/backend/imageanalysis/infrastructure/OpenAiClient.java (2)
30-30: import 문의 wildcard 사용이 적절합니다.여러 ErrorCode 상수를 사용하고 있어 wildcard import가 코드 가독성을 향상시킵니다.
205-205: 이미지 URL 전처리 로직이 적절히 추가되었습니다.
trimImageUrl호출로 이미지 URL의 유효성을 보장하여 안정성이 향상되었습니다.
Comment on lines
+240
to
+251
| private String trimImageUrl(String url) { | ||
| for (ImageMimeType type : ImageMimeType.values()) { | ||
| String extension = "." + type.name().toLowerCase(); | ||
| int index = url.toLowerCase().indexOf(extension); | ||
| if (index != -1) { | ||
| int endIndex = index + extension.length(); | ||
| return url.substring(0, endIndex); | ||
| } | ||
| } | ||
| throw new BusinessException(NO_IMAGE_FILE_EXTENSION_FOUND); | ||
| } | ||
|
|
There was a problem hiding this comment.
🛠️ Refactor suggestion
trimImageUrl 메서드 구현을 개선할 수 있습니다.
현재 구현은 기본적으로 작동하지만 몇 가지 개선사항이 있습니다:
- 성능 최적화: 모든 ImageMimeType을 순회하는 대신 더 효율적인 방법 사용
- 정확성 향상: 현재는 URL 어디서든 확장자를 찾지만, URL의 path 부분에서만 찾는 것이 더 정확
- 코드 간소화: 로직을 더 간결하게 작성 가능
다음과 같이 개선을 제안합니다:
private String trimImageUrl(String url) {
- for (ImageMimeType type : ImageMimeType.values()) {
- String extension = "." + type.name().toLowerCase();
- int index = url.toLowerCase().indexOf(extension);
- if (index != -1) {
- int endIndex = index + extension.length();
- return url.substring(0, endIndex);
- }
- }
- throw new BusinessException(NO_IMAGE_FILE_EXTENSION_FOUND);
+ try {
+ // URL에서 path 부분만 추출 (쿼리 파라미터 제거)
+ String urlPath = url.split("\\?")[0];
+
+ // path에서 파일명 추출
+ String fileName = urlPath.substring(urlPath.lastIndexOf('/') + 1);
+
+ // 확장자 추출
+ int dotIndex = fileName.lastIndexOf('.');
+ if (dotIndex == -1) {
+ throw new BusinessException(NO_IMAGE_FILE_EXTENSION_FOUND);
+ }
+
+ String extension = fileName.substring(dotIndex + 1).toLowerCase();
+
+ // ImageMimeType에서 지원하는 확장자인지 확인
+ boolean isValidExtension = Arrays.stream(ImageMimeType.values())
+ .anyMatch(type -> type.name().toLowerCase().equals(extension));
+
+ if (!isValidExtension) {
+ throw new BusinessException(UNSUPPORTED_IMAGE_TYPE);
+ }
+
+ // 확장자까지만 포함하는 URL 반환
+ int extensionEndIndex = urlPath.lastIndexOf('.') + extension.length() + 1;
+ return url.substring(0, extensionEndIndex);
+
+ } catch (Exception e) {
+ if (e instanceof BusinessException) {
+ throw e;
+ }
+ throw new BusinessException(NO_IMAGE_FILE_EXTENSION_FOUND);
+ }
}이렇게 하면:
- URL의 path 부분에서만 확장자를 찾아 더 정확함
- 쿼리 파라미터가 있어도 올바르게 처리됨
- 지원하지 않는 확장자에 대해서도 적절한 에러 처리
- 성능 개선 (순회 대신 직접 확장자 추출)
Committable suggestion skipped: line range outside the PR's diff.
🤖 Prompt for AI Agents
In
src/main/java/com/webeye/backend/imageanalysis/infrastructure/OpenAiClient.java
around lines 240 to 251, improve the trimImageUrl method by extracting the file
extension only from the URL's path segment instead of searching the entire URL
string. Parse the URL to isolate the path, then check if it ends with any
supported ImageMimeType extension. Return the substring up to the extension's
end if found, otherwise throw the BusinessException. This approach enhances
accuracy by ignoring query parameters, improves performance by avoiding full
iteration over all types, and simplifies the code logic.
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.
#️⃣ 연관된 이슈
📝 작업 내용
trimImageUrl()구현📸 스크린샷
💬 리뷰 요구사항
Summary by CodeRabbit