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 streaming support #5

Merged
merged 1 commit into from
Mar 31, 2024
Merged

Add streaming support #5

merged 1 commit into from
Mar 31, 2024

Conversation

bep
Copy link
Owner

@bep bep commented Mar 22, 2024

No description provided.

@bep bep force-pushed the feat/stream branch 20 times, most recently from 4efcef6 to 122ca36 Compare March 27, 2024 09:37
@bep bep marked this pull request as ready for review March 28, 2024 09:34
@bep bep force-pushed the feat/stream branch 9 times, most recently from c6db832 to 426e817 Compare March 31, 2024 13:30
@bep bep force-pushed the feat/stream branch 3 times, most recently from 2ddcabe to 3ad6a67 Compare March 31, 2024 16:23
The old request/response setup worked great, but had its limitations.

The new model types are request, message (zero or more) and a receipt.

The receipt can immplement some optional interfaces that gets/sets

* ETag
* ELastModified
* ESize

These may autimatically be set by the library if not provided by user (the `Etag` bassed on all message bodies if there's a hash function configured).

The primary use case for the above would be client side caching.

This also removes the slow Gob codec.
@bep bep merged commit 5f5b44b into main Mar 31, 2024
6 checks passed
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