-
Notifications
You must be signed in to change notification settings - Fork 8
chore: gradle 의존성 정리 #279
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: gradle 의존성 정리 #279
Conversation
- QClass는 '수정할 수 있는 소스코드'가 아니므로, 소스코드에서 지웁니다. - 빌드될 때 기본 위치(/build/generated/sources/annotationProcessor/java/main)에 자동으로 생성되게 합니다.
Walkthrough
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
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
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
build.gradle (1)
71-74: 3. 소스셋 구성 리뷰:
1. 자동 생성 소스 포함:
sourceSets에build/generated/sources/annotationProcessor/java/main디렉토리를 추가하여, 빌드 시 자동 생성된 QueryDSL 소스 파일들이 정상적으로 컴파일에 포함되도록 설정한 점이 좋습니다.
2. 오타 수정 제안:
주석에// To include QueryDLS classes in compile classpath라고 되어 있는데, 올바른 표기는QueryDSL입니다. 해당 오타를 수정하면 더욱 깔끔할 것 같습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (12)
src/main/generated/com/example/solidconnection/application/domain/QApplication.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/application/domain/QGpa.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/application/domain/QLanguageTest.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/entity/QCountry.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/entity/QInterestedCountry.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/entity/QInterestedRegion.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/entity/QRegion.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/siteuser/domain/QSiteUser.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/university/domain/QLanguageRequirement.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/university/domain/QLikedUniversity.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/university/domain/QUniversity.javais excluded by!**/generated/**src/main/generated/com/example/solidconnection/university/domain/QUniversityInfoForApply.javais excluded by!**/generated/**
📒 Files selected for processing (1)
build.gradle(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build
🔇 Additional comments (2)
build.gradle (2)
24-65: 1. 종속성 섹션 리뷰:
1. Flyway 의존성 추가:
Flyway 관련 의존성인org.flywaydb:flyway-core와org.flywaydb:flyway-mysql이 추가되어 데이터베이스 마이그레이션 관리가 향상되었습니다.
2. QueryDSL 의존성 변경:
보안 취약점(CVE-2024-49203)을 해결하기 위해 기존 QueryDSL 대신io.github.openfeign.querydsl:querydsl-jpa:6.11및 관련 annotationProcessor 의존성이 적용되었습니다.
3. Spring Security 의존성 추가:
org.springframework.security:spring-security-config와org.springframework.security:spring-security-web의존성이 추가되어 보안 관련 설정이 강화되었습니다.
4. Lombok 의존성 개선:
compileOnly 'org.projectlombok:lombok'에서 버전 지정이 제거되어, Spring Boot가 의존성 관리를 수행하도록 개선되었습니다.
5. 기타 의존성 수정:
Hibernate Validator와 AWS SDK 의존성이 각각org.hibernate.validator:hibernate-validator와com.amazonaws:aws-java-sdk-s3:1.12.782로 추가/수정되어 최신 보안 및 기능 지원이 보장됩니다.전체적으로 Gradle 의존성 정리 작업이 Spring Boot의 자동 의존성 관리 원칙에 부합하도록 잘 구성된 것으로 보입니다.
67-69: 2. 테스트 태스크 구성 리뷰:
1. JUnit 플랫폼 사용:
tasks.named('test', Test) { useJUnitPlatform() }설정을 통해 테스트 실행 시 최신 JUnit 플랫폼을 활용하도록 구성되어 있는 점이 긍정적입니다.이 구성이 최신 테스트 프레임워크의 기능을 온전히 활용할 수 있도록 돕습니다.
|
cf. coderabbit 이 말하는거 거짓말입니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
스프링 부트가 버전을 알아서 관리해주는군요. 좋습니다~
Gyuhyeok99
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3번 너무 좋네요 ㅎㅎ 고생하셨습니다!
관련 이슈
작업 내용
스프링에서 버전을 관리해주는 의존성은 버전을 지웠습니다.
우리가 쓰는 3.1.5는 링크의 버전을 자동으로 사용한다고 합니다.
버전을 명시하고 우리가 수동으로 관리나 최적화해야 할 것은 없는 듯 하여,..
스프링 부트에서 자동으로 관리하게 했습니다.
com.querydsl 그룹의 query dls 의존성이 보안 취약점(CVE-2024-49203)이 있어
io.github.openfeign.querydsl 그룹으로 바꿨습니다.
ref. https://dev-gallery.tistory.com/61
소스코드로 올라와있는 qclass를 지우고, 빌드 과정에서 생성되게 했습니다.
빌드될 때 기본 위치(/build/generated/sources/annotationProcessor/java/main)에 생성되고,
컴파일 시 포함되게 했습니다!
중복되거나 불필요한 의존성 삭제했습니다.
어떻게 했냐면,..
하나하나 없애고 clean하고 build하면서 오류 안나나 보고..
의존성 검색해가면서 불필요한 것들은 없앴습니다 👴🏻👵🏻
특이 사항
리뷰 요구사항 (선택)