runtime icon indicating copy to clipboard operation
runtime copied to clipboard

[Http Retry] Response headers interference

Open joepkalthoff opened this issue 1 year ago • 0 comments

Summary

Retries in the Http component include headers from the previous (re)try.

Testcase

Dovetail version: 4.17.0-SNAPSHOT-dcb7cc8 Instance: next Tenant: Regression Tests

Sending Flow: HttpRetrySocketAsynchronous Flow version: 7 url: https://next.dovetail.world/flowdesigner/66b1efbde56ba4001000012e/7/route

Receiving Flow: HttpRetry500 Flow version: 1 url: https://next.dovetail.world/flowdesigner/66b31c0ee56ba4000e00002f/1/route

Input:

  • Make sure the flow is installed in test and kick off the flow: https://next.dovetail.world/test/inbound_http/regressiontests/HttpRetrySocketAsynchronous

Output: Notice the transactions of the Sending flow and the Receiving flow. Especially the body and headers for the first try and the 2 retries. There are screenshots in the results section below:

Result

Initial Http request

Sending Image

Receiving Image

1st retry

Sending Image

Receiving Image

2nd retry

Sending Image

Receiving Image

After the Http retries

Image

Conclusion

I expect the headers from the 5xx responses to not interfere with the retry messages. Only the breadcrumbId, Content-Type and counter header should be included.

There should be a way to see the response headers somewhere though. They can contain valuable information about why the server responded with a 5xx in the first place. Maybe in the Logs of a flow?

The same is true for the 5xx response body. This is a seperate issue: https://github.com/assimbly/runtime/issues/268.

Original issue on Dovetail Application board: https://github.com/dovetailworld/front-end/issues/4635.

joepkalthoff avatar Sep 04 '24 10:09 joepkalthoff