request-baskets icon indicating copy to clipboard operation
request-baskets copied to clipboard

Allow for storing and displaying responses

Open nextrevision opened this issue 7 years ago • 4 comments

Optionally allow for storing and displaying response data like headers and content. I'm using this with the response proxy. Not sure if you want to support something like this, but if so, I can polish up my fork and submit a PR.

nextrevision avatar May 03 '18 15:05 nextrevision

This make sense if combined with issue #24, there is already a structure to hold the Response defined in the API. The Request may optionally contain a response property then.

Probably another boolean flag to enable storing the responses would make sense in the settings, or it should be implicitly enabled, when basket operates in "full proxy" mode, and disabled otherwise.

I'm open to discuss the options here :)

darklynx avatar May 03 '18 21:05 darklynx

I'll add the boolean flag, since storing the responses can run up the memory when using the in-memory store. I currently have the response nested in a request struct, but will play around with the data structure.

nextrevision avatar May 04 '18 13:05 nextrevision

I guess its fine to encapsulate response in request, otherwise the API break should take place: I mean the external REST API

darklynx avatar May 04 '18 14:05 darklynx

What about this? Can we already do this, or is this still under consideration?

DustRaven avatar Mar 07 '23 10:03 DustRaven