juspay/hyperswitch

[BUG] 5xx while parsing `Evidence Type` in Disputes Attach Evidence file upload flow

Open

#3,489 opened on Jan 30, 2024

View on GitHub
 (3 comments) (0 reactions) (2 assignees)Rust (4,676 forks)batch import
A-coreC-buggood first issue

Repository metrics

Stars
 (42,690 stars)
PR merge metrics
 (Avg merge 6d 22h) (213 merged PRs in 30d)

Description

Bug Description

5xx while parsing Evidence Type in Disputes File upload flow

Expected Behavior

It should have been a Bad request, telling Invalid Evidence Type

Actual Behavior

We are throwing 5xx

Steps To Reproduce

Provide an invalid evidence_type while doing File upload in Dispute Attach Evidence flow

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

  • I checked and didn't find a similar issue

Have you read the Contributing Guidelines?

Are you willing to submit a PR?

None

Contributor guide

Research direction
Investigate the file upload handler in the disputes module to find where `Evidence Type` is parsed. Add proper error handling to return a 400 Bad Request for invalid evidence types instead of a 5xx server error.
Tech stack
rust
Domain
backend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Rust
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70