starred/mersel-dss-server-signer-java
1
0
Fork 0
mirror of https://github.com/mersel-dss/mersel-dss-server-signer-java.git synced 2026-04-26 16:25:48 +03:00
Code Issues 4 Projects Releases Packages Wiki Activity

[PR #6] [MERGED] [FEATURE](https://github.com/mersel-dss/mersel-dss-server-signer-java… #9

New issue
Closed
opened 2026-03-02 01:54:02 +03:00 by kerem · 0 comments
kerem commented 2026-03-02 01:54:02 +03:00
Owner
Copy link

📋 Pull Request Information

Original PR: https://github.com/mersel-dss/mersel-dss-server-signer-java/pull/6
Author: @hasanyildiz
Created: 11/13/2025
Status: ✅ Merged
Merged: 11/16/2025
Merged by: @erdembas

Base: main ← Head: feat/slot_ve_slotlistindex_ile_calisma

📝 Commits (1)

📊 Changes

5 files changed (+30 additions, -14 deletions)

View changed files

📝 src/main/java/io/mersel/dss/signer/api/SignatureApplication.java (+7 -5)
📝 src/main/java/io/mersel/dss/signer/api/config/SignatureConfiguration.java (+2 -1)
📝 src/main/java/io/mersel/dss/signer/api/controllers/CertificateInfoController.java (+2 -1)
📝 src/main/java/io/mersel/dss/signer/api/models/configurations/SignatureServiceConfiguration.java (+7 -0)
📝 src/main/java/io/mersel/dss/signer/api/services/keystore/PKCS11KeyStoreProvider.java (+12 -7)

📄 Description

…/issues/4)

Provider slotListIndex ve slot parametrelerinin ayrı ayrı kullanılabilmesi sağlandı.

📝 Değişiklik Açıklaması

slotListIndex ve slot parametrelerinin ayrı ayrı kullanılabilmesi

🔗 İlgili Issue

Fixes #(4)

🎯 Değişiklik Türü

  • 🐛 Bug fix (geriye uyumlu hata düzeltmesi)
  • ✨ Yeni özellik (geriye uyumlu yeni işlevsellik)
  • 💥 Breaking change (geriye uyumsuz değişiklik)
  • 📝 Dokümantasyon güncellesi
  • ♻️ Refactoring (davranış değişikliği olmadan kod iyileştirme)
  • ⚡ Performance iyileştirmesi
  • 🧪 Test ekleme/güncelleme

🧪 Testler

  • Yeni unit testler eklendi
  • Mevcut testler güncellendi
  • Tüm testler başarılı (mvn test)
  • Manuel olarak test edildi

📋 Kontrol Listesi

  • Kod standartlarına uygun (Javadoc, isimlendirme)
  • Commit mesajları açıklayıcı
  • CHANGELOG.md güncellendi (major değişiklikler için)
  • README.md güncellendi (gerekirse)
  • Linter hataları yok
  • Breaking change ise migration guide eklendi

💭 Ek Notlar

Reviewer'ların bilmesi gereken başka bir şey var mı?

ENV PKCS11_SLOT -> provider config içerisinde 'slot'
ENV PKCS11_SLOT_LIST_INDEX-> provider config içerisinde 'slotListIndex'
değerine karşılık gelecek şekilde düzenleme yapıldı. Aynı anda ikisi birden tanımlı ve geçerli(>=0) olması halinde öncelikle slotListIndex dikkate alınacak şekilde tanımlandı.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/mersel-dss/mersel-dss-server-signer-java/pull/6 **Author:** [@hasanyildiz](https://github.com/hasanyildiz) **Created:** 11/13/2025 **Status:** ✅ Merged **Merged:** 11/16/2025 **Merged by:** [@erdembas](https://github.com/erdembas) **Base:** `main` ← **Head:** `feat/slot_ve_slotlistindex_ile_calisma` --- ### 📝 Commits (1) - [`9e62eb0`](https://github.com/mersel-dss/mersel-dss-server-signer-java/commit/9e62eb04f4019eef4f4090a4739e0c2f56c99ce1) [FEATURE](https://github.com/mersel-dss/mersel-dss-server-signer-java/issues/4) ### 📊 Changes **5 files changed** (+30 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `src/main/java/io/mersel/dss/signer/api/SignatureApplication.java` (+7 -5) 📝 `src/main/java/io/mersel/dss/signer/api/config/SignatureConfiguration.java` (+2 -1) 📝 `src/main/java/io/mersel/dss/signer/api/controllers/CertificateInfoController.java` (+2 -1) 📝 `src/main/java/io/mersel/dss/signer/api/models/configurations/SignatureServiceConfiguration.java` (+7 -0) 📝 `src/main/java/io/mersel/dss/signer/api/services/keystore/PKCS11KeyStoreProvider.java` (+12 -7) </details> ### 📄 Description …/issues/4) Provider slotListIndex ve slot parametrelerinin ayrı ayrı kullanılabilmesi sağlandı. ## 📝 Değişiklik Açıklaması slotListIndex ve slot parametrelerinin ayrı ayrı kullanılabilmesi ## 🔗 İlgili Issue Fixes #([4](https://github.com/mersel-dss/mersel-dss-server-signer-java/issues/4)) ## 🎯 Değişiklik Türü - [ ] 🐛 Bug fix (geriye uyumlu hata düzeltmesi) - [x] ✨ Yeni özellik (geriye uyumlu yeni işlevsellik) - [ ] 💥 Breaking change (geriye uyumsuz değişiklik) - [ ] 📝 Dokümantasyon güncellesi - [ ] ♻️ Refactoring (davranış değişikliği olmadan kod iyileştirme) - [ ] ⚡ Performance iyileştirmesi - [ ] 🧪 Test ekleme/güncelleme ## 🧪 Testler - [ ] Yeni unit testler eklendi - [ ] Mevcut testler güncellendi - [ ] Tüm testler başarılı (`mvn test`) - [ ] Manuel olarak test edildi ## 📋 Kontrol Listesi - [x] Kod standartlarına uygun (Javadoc, isimlendirme) - [x] Commit mesajları açıklayıcı - [ ] CHANGELOG.md güncellendi (major değişiklikler için) - [ ] README.md güncellendi (gerekirse) - [x] Linter hataları yok - [ ] Breaking change ise migration guide eklendi ## 💭 Ek Notlar Reviewer'ların bilmesi gereken başka bir şey var mı? ENV PKCS11_SLOT -> provider config içerisinde 'slot' ENV PKCS11_SLOT_LIST_INDEX-> provider config içerisinde 'slotListIndex' değerine karşılık gelecek şekilde düzenleme yapıldı. Aynı anda ikisi birden tanımlı ve geçerli(>=0) olması halinde öncelikle slotListIndex dikkate alınacak şekilde tanımlandı. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-02 01:54:02 +03:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
pr-11-CadesSignDevelope
pr-10-UBLExtensionElement
No results found.
Labels
Clear labels   
bug

   
enhancement

   
enhancement

   
pull-request

Mirrored from GitHub Pull Request

No labels
bug
enhancement
enhancement
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/mersel-dss-server-signer-java#9
No description provided.