platform icon indicating copy to clipboard operation
platform copied to clipboard

fix:Incorrect response received when space is added before `requestId` in the requestId parameter

Open sayali-chavan2396 opened this issue 9 months ago • 1 comments

Description: When initiating the bulk issuance process using the Start bulk-issuance API, providing a space before the requestId parameter results in an incorrect error response

Method

POST/v1/orgs/{orgId}/{requestId}/bulk (bulk issue credential)

Request

Input: Provide space before requestId in requestId param [ requestId : 21d12b17-46c6-411a-b6cb-e8a015da38ce ]

Steps to Reproduce

  1. Execute the bulk-issuance API.
  2. Add a space before the requestIdIncorrect response when space is included before requestId in the requestId parameter
  3. Observe the response.

Current behavior { "statusCode": 400, "message": "Timeout for reviewing data, re-upload your file and generate new request", "error": "Bad Request" }

Expected behavior

Trim space and return successful response { "statusCode": 201, "message": "Issuance process started. It will take some time", "data": "Process initiated for bulk issuance" }

Screenshot

Image

sayali-chavan2396 avatar May 08 '25 10:05 sayali-chavan2396

@GHkrishna I have reported an issue. Please review it and let me know if you need any further information from my side. Thanks!

sayali-chavan2396 avatar May 08 '25 10:05 sayali-chavan2396

Hi team, I would like to work on this issue.

sr467113 avatar May 28 '25 16:05 sr467113

Hello @sr467113 You can start working on it, assigning you the issue

GHkrishna avatar May 30 '25 05:05 GHkrishna

Submitted. #1257 Thanks GHkrishna

sr467113 avatar Jun 01 '25 11:06 sr467113

@sayali-chavan2396 Can we close this?

GHkrishna avatar Jul 11 '25 05:07 GHkrishna

@GHkrishna I have retested the above-mentioned issue, and it is now working as expected. Therefore, I am closing this issue.

Current Response { "statusCode": 201, "message": "Issuance process started. It will take some time", "data": "Process initiated for bulk issuance" }

sayali-chavan2396 avatar Jul 11 '25 06:07 sayali-chavan2396