Ability to add response body to httpClientIntegration

[jerbob92]

Problem Statement

It would be very useful to have the response body for errors, because it mostly gives more explanation what caused an error.

Solution Brainstorm

Add two extra options to httpClientIntegration:

• addRequestBody
• addResponseBody

[mydea]

Hey, thanks for writing in.

Sadly, capturing payloads like this is relatively complex, there are a bunch of edge cases etc. to handle when trying to solve this generically 😬

So instead, what would you think about adding a new option to the httpClientIntegration that allows to configure a callback where you can do stuff yourself, e.g.:

httpClientIntegration({
  beforeCaptureEvent(event, data) {
    if(data.xhr.response) {
      // put response on event...
    } else if (data.fetch.response) {
      // put response on event...
   }
  }
})

[lforst]

I think this would be very valuable to add as an opt-in option.

[jerbob92]

@mydea

So instead, what would you think about adding a new option to the httpClientIntegration that allows to configure a callback where you can do stuff yourself, e.g.:

I think that would be fine as well, I have currently implemented it by making a custom HTTP_INTERCEPTORS class (Angular) that basically does this.

[mydea]

I'll put this in our backlog as something todo - we are pretty busy right now, so PRs are always welcome as well! :)