juspay/hyperswitch

[REFACTOR]: [Globalpay] MCA metadata validation

Open

#2 911 ouverte le 17 nov. 2023

Voir sur GitHub
 (6 commentaires) (0 réactions) (1 assigné)Rust (4 676 forks)batch import
A-connector-integrationC-refactorgood first issuehelp wanted

Métriques du dépôt

Stars
 (42 690 stars)
Métriques de merge PR
 (Merge moyen 8j 12h) (212 PRs mergées en 30 j)

Description

:memo: Feature Description

  • We don't have validation for MCA metadata, So while creating MCA we are allowing any json value. Because of which RequestEncodingFailure happening during payments for the MCA which doesn't have the expected fields

:hammer: Possible Implementation

  • While adding Merchant Connector Account we want that metadata field should be validated.
  • You can check this PR for further reference https://github.com/juspay/hyperswitch/pull/2746 :bookmark: Note: All the changes needed should be contained within hyperswitch/crates/router/src/connector/globalpay and crates/router/src/core/admin.rs

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

  • I checked and didn't find a similar issue

:package: Have you read the Contributing Guidelines?

:sparkles: Are you willing to submit a PR?

Guide contributeur

Direction de recherche
Validez les métadonnées MCA dans le connecteur Globalpay en ajoutant une logique de validation dans crates/router/src/connector/globalpay et crates/router/src/core/admin.rs. Référencez la PR #2746 pour une implémentation similaire.
Stack technique
rust
Domaine
backendapipayments
Type d'issue
Fonctionnalité
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