juspay/hyperswitch

[REFACTOR] Use `MerchantJwtWithProfileId` instead of `MerchantJwt` in Authentication data

Open

#5 937 ouverte le 18 sept. 2024

Voir sur GitHub
 (13 commentaires) (0 réactions) (1 assigné)Rust (4 676 forks)batch import
A-usersC-refactorgood first issuehacktoberfest

Métriques du dépôt

Stars
 (42 690 stars)
Métriques de merge PR
 (Merge moyen 6j 22h) (213 PRs mergées en 30 j)

Description

Feature Description

We have introduced MerchantJwtWithProfileId to the AuthenticationData enum. The new variant includes both a profile_id and a non-optional user_id field, improving upon the older MerchantJwt variant. The goal of this issue is to replace all instances of MerchantJwt with MerchantJwtWithProfileId.

You can find the location of MerchantJwt here: https://github.com/juspay/hyperswitch/blob/1ff66a720b29a169870b2141e6ffd585b4d5b640/crates/router/src/services/authentication.rs#L128 MerchantJwtWithProfileId is located just under it.

Possible Implementation

  • Refactor the existing codebase to replace occurrences of MerchantJwt with MerchantJwtWithProfileId.
  • Ensure that merchant_id, user_id, and profile_id (where necessary) are properly handled in the updated variant.

Have you spent some time checking if this feature request has been raised before?

  • I checked and didn't find a similar issue

Submission Process:

  • Ask the maintainers for assignment of the issue, you can request for assignment by commenting on the issue itself.
  • Once assigned, submit a pull request (PR).
  • Maintainers will review and provide feedback, if any.
  • Maintainers can unassign issues due to inactivity, read more here.

Refer here for Terms and conditions for the contest.

Guide contributeur

Direction de recherche
Tout d'abord, localisez la variante `MerchantJwt` dans `crates/router/src/services/authentication.rs` vers la ligne 128. Ensuite, recherchez toutes les occurrences de `MerchantJwt` dans l'ensemble de la base de code. Remplacez chacune par `MerchantJwtWithProfileId`, en veillant à fournir les champs requis (profile id, user id non optionnel). Confirmez que le constructeur de la nouvelle variante est utilisé correctement. Vérifiez que le fichier `authentication.rs` compile et que les tests associés passent. Aucune PR liée ou commentaires supplémentaires ne clarifient des cas particuliers spécifiques ; envisagez de vérifier la définition de l'énumération et son utilisation dans d'autres modules.
Stack technique
rust
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75