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.

Sign up for GitHub

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

Jump to bottom

Calling Send() with large objects errors out in the spdy implementation #65

Open
BrianBland opened this issue Oct 25, 2014 · 0 comments
Open

Calling Send() with large objects errors out in the spdy implementation #65

BrianBland opened this issue Oct 25, 2014 · 0 comments
Labels
bug
Milestone
v0.2.0

Comments

@BrianBland
Copy link

The spdy channel.Send method will return a spdy.InvalidDataFrame error if the serialized size of the message is greater than 16MB (the spdy max data frame size). This could be fixed with a combination of automatic data frame chunking (moby/spdystream#31) and using the underlying spdystream object as an io.Reader for msgpack deserialization on the Receive side.
@BrianBland BrianBland added the bug label Oct 25, 2014
BrianBland added a commit to BrianBland/docker-registry that referenced this issue Oct 25, 2014
@BrianBland
Uses streams internally for ipc Get/Put Content
1c68bdb 
This is done because libchan/spdystream does not currently support sending
serialzied objects of size larger than 16MB
See docker/libchan#65
BrianBland added a commit to BrianBland/docker-registry that referenced this issue Oct 25, 2014
@BrianBland
Uses streams internally for ipc Get/Put Content
7c892de 
This is done because libchan/spdystream does not currently support sending
serialzied objects of size larger than 16MB
See docker/libchan#65
@dmcgowan dmcgowan added this to the v0.2.0 milestone Feb 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
v0.2.0
Development

No branches or pull requests
2 participants
@dmcgowan @BrianBland