SendHandler does not populate the HTTPResponse

[odannyc]

This library is not adding the HTTPResponse once we send off the request via the SendHandler. Look at the original aws-sdk-go for an example:

https://github.com/aws/aws-sdk-go/blob/main/aws/corehandlers/handlers.go#L112

[odannyc]

@ktseytlin @kevinchristen

Can you guys provide some input on this?

I have other handlers that rely on HTTPResponse to be there and unfortunately this is not including it.

related:
newrelic/go-agent#287

[kevinchristen]

DAX does not use HTTP. Can you give us more information about how your application relies on the response?

[odannyc]

Thank you for responding back. There is a library (NewRelic) I'm using for tracing that relies on that field to be populated. Here is the code:

https://github.com/newrelic/go-agent/blob/master/_integrations/nrawssdk/v1/nrawssdk.go#L26-L29