Skip to content

Conversation

@hachi029
Copy link
Contributor

@hachi029 hachi029 commented Nov 26, 2025

Description

when there is any timeout sub request, the number of sub responses does not match the number of sub requests。

for example:

the batch-request with 2 sub requests
{"headers":{},"pipeline":[{"path":"/ok","method":"GET"},{"path":"/timeout","method":"GET"}]}

there are 3 sub responses in the batch-response, with an extra empty json object '{}'
[{"status":200,"body":"ok","reason":"OK","headers":{"Connection":"keep-alive"}},{"status":504,"reason":"upstream timeout"},{}]

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working labels Nov 26, 2025
@hachi029 hachi029 changed the title batch-request: the number of sub responses does not match the number of sub requests fix: (batch-requests)the number of sub responses does not match the number of sub requests Nov 27, 2025
@hachi029 hachi029 changed the title fix: (batch-requests)the number of sub responses does not match the number of sub requests fix(batch-requests): the number of sub responses does not match the number of sub requests Nov 27, 2025
@hachi029 hachi029 changed the title fix(batch-requests): the number of sub responses does not match the number of sub requests fix(batch-requests): the number of sub-responses does not match that of sub-requests Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants