-
Notifications
You must be signed in to change notification settings - Fork 2
Updated pom.xml by Safer #175
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
base: main
Are you sure you want to change the base?
Conversation
WalkthroughBu değişiklikte, projenin 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
|
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: 1
🔭 Outside diff range comments (2)
pom.xml (2)
27-33
: Çakışanorg.projectlombok.version
tanımları derleme sırasında belirsizlik yaratıyor
<org.projectlombok.version>
hem 27. satırda1.18.24
, hem de 32. satırda1.18.26
olarak tanımlanmış. Maven, son görünen tanımı kullanır; ancak bu durum okuyucuda kafa karışıklığına ve beklenmedik sürüm drift’lerine yol açar.- <org.projectlombok.version>1.18.24</org.projectlombok.version> ... - <org.projectlombok.version>1.18.26</org.projectlombok.version> + <!-- Tek bir net tanım bırakın --> + <org.projectlombok.version>1.18.26</org.projectlombok.version>
21-23
: Property değerleri ile bağımlılık sürümleri uyuşmuyor (firebase-admin
,jackson-dataformat
vb.)Örneğin:
- Property
firebase-admin.version
→9.2.0
, ancak bağımlılık satırında9.5.0
- Property
com.fasterxml.jackson.dataformat.version
→2.14.2
, ancak bağımlılık satırında2.15.0
Bu tutarsızlıklar, property’lerin artık kullanılmadığını veya güncelleme aracının incomplete çalıştığını gösterir.
Ya property’leri güncelleyin ya da kullanılmayanları tamamen kaldırın; aksi halde bakım maliyeti artar.Also applies to: 37-38
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
pom.xml
(5 hunks)
🔇 Additional comments (1)
pom.xml (1)
230-238
: AnnotationProcessorPaths hâlâ property referanslarına bağlı; yukarıdaki tutarsızlıklar doğrudan burayı da etkilerEğer property değerleri güncellenmezse, derleyici
mapstruct
velombok
annotation processor’larında farklı sürümler kullanabilir. Yukarıdaki property temizlik/güncelleme işlemi sonrası bu bölümün hâlâ doğru sürümleri işaret ettiğini doğrulayın.
<artifactId>spring-boot-devtools</artifactId> | ||
<optional>true</optional> | ||
<version>${org.springframework.boot.version}</version> | ||
<version>3.3.12</version> | ||
</dependency> | ||
<dependency> | ||
<groupId>org.springframework.boot</groupId> | ||
<artifactId>spring-boot-starter-web</artifactId> | ||
<version>${org.springframework.boot.version}</version> | ||
<version>3.3.13</version> | ||
</dependency> |
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.
🛠️ Refactor suggestion
Spring Boot starter’larda sabit sürüm tanımlamak parent BOM ile çelişiyor
spring-boot-starter-*
bağımlılıklarına (devtools
, web
, data-jpa
, validation
) manuel olarak 3.3.12/3.3.13
sürümü verilmiş. Proje parent’ı hâlen 3.1.4
; bu durumda:
- Versiyon çakışmaları ve transitif bağımlılık uyumsuzlukları oluşur.
- Spring Boot’un BOM mekanizmasının sağladığı güvenli sürüm seti devre dışı kalır.
En basit ve güvenli yol, starter bağımlılıklarındaki <version>
etiketlerini tamamen silip parent sürümünüzü tek kaynak hâline getirmektir.
- <version>3.3.12</version>
+ <!-- Spring Boot parent BOM'u versiyonu yönetecek -->
Aynı işlemi diğer starter’lara da uygulayın.
Also applies to: 81-84, 134-138
🤖 Prompt for AI Agents
In pom.xml around lines 59 to 67, 81 to 84, and 134 to 138, the
spring-boot-starter dependencies have explicit version tags that conflict with
the parent BOM version 3.1.4. Remove the <version> tags from these starter
dependencies (devtools, web, data-jpa, validation) to rely solely on the parent
BOM for version management, ensuring consistent and compatible dependency
versions.
This PR was automatically created by Safer, an open-source tool that updates vulnerable dependencies with compatible and more secure versions.
Analyzed commit: c9e49b7
File updated: pom.xml
Vulnerabilities reduced: 163 -> 79
Let us know if you have questions.
Thanks,
Safer Bot
Summary by CodeRabbit