Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add response data streaming facility #85

Closed
wants to merge 1 commit into from

Conversation

rsjethani
Copy link

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
Author

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

1 similar comment
@rsjethani
Copy link
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