close
Skip to content

Add response data streaming facility#85

Closed
rsjethani wants to merge 1 commit into
dghubble:mainfrom
rsjethani:add-bytestreamer
Closed

Add response data streaming facility#85
rsjethani wants to merge 1 commit into
dghubble:mainfrom
rsjethani:add-bytestreamer

Conversation

@rsjethani
Copy link
Copy Markdown

Here we add a new ResponseDecoder: ByteStreamer. This allows user to get a stream of bytes 'as is' i.e. without any decoding. This is useful for all those streaming use cases like downloading files, binary blobs etc.

This is also useful when response data is just plain text for e.g. the 5XX responses from API gateways.

Side changes:

  • The xml stuff in testing code no longer needed as this new decoder is used as non-default decoder.

  • Code reformatting as per gopls/gofumpt style.

Here we add a new ResponseDecoder: ByteStreamer. This allows user to get
a stream of bytes 'as is' i.e. without any decoding. This is useful for
all those streaming use cases like downloading files, binary blobs etc.

This is also useful when response data is just plain text for e.g. the
5XX responses from API gateways.

Side changes:
- The xml stuff in testing code no longer needed as this new
decoder is used as non-default decoder.

- Code reformatting as per gopls/gofumpt style.
@rsjethani
Copy link
Copy Markdown
Author

Closing as changes are now part of newly maintained hard fork https://github.com/rsjethani/rsling

1 similar comment
@rsjethani
Copy link
Copy Markdown
Author

Closing as changes are now part of newly maintained hard fork https://github.com/rsjethani/rsling

@rsjethani rsjethani closed this Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant