ADORSYS-GIS / ADORSYS-GIS/keycloak-oid4vp-plugin

Deduplicate wallet attestation validation code

Fermée
#197 0 commentaires 0 réactions 1 personne assignée Réclamée par @Ogenbertrand Voir sur GitHub
Langage dominant
Java
Étoiles
35
Forks
1
Merge moyen
2 j 9 h
PR mergées (30 j)
17

Description

Class `WalletAttestationValidator` appears to duplicate code already existing in upstream Keycloak for the same purpose.
Investigate and reuse instead.

https://github.com/ADORSYS-GIS/keycloak-oid4vp-plugin/blob/7a9bbe2c0fb5052bf01dc17812ba21d2378ec116/src/main/java/io/github/adorsysgis/keycloak/protocol/oid4vc/presentation/WalletAttestationValidator.java#L52-L58

ETA: 2 days

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Start in src/main/java/io/github/adorsysgis/keycloak/protocol/oid4vc/presentation/WalletAttestationValidator.java, especially lines 52-58, and compare the validation code with the corresponding upstream Keycloak implementation. Confirm whether the upstream code can be reused without changing wallet-attestation behavior, then run the project's existing tests to verify the refactor.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
authentication
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
Active
Clarté
Plutôt claire
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.