hyperswitch icon indicating copy to clipboard operation
hyperswitch copied to clipboard

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

Open ThisIsMani opened this issue 5 months ago • 6 comments

Feature Description

We have recently 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/2bc8756e061e9e0e705ce2c6b533c3583f3efa8b/crates/router/src/services/authentication.rs#L91 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?

  • [X] 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.

ThisIsMani avatar Sep 18 '24 07:09 ThisIsMani