Commit Briefs
Fix EOF indication from Readdir
EOF is indicated by a zero-length result with no error, so ensure that io.EOF does not ever get returned by Readdir.Read. Signed-off-by: Jonathan Rudenberg <jonathan@titanous.com>
Clean up logging
Ensure that the server only logs in exceptional cases and prefixes each log line with the package name. Signed-off-by: Jonathan Rudenberg <jonathan@titanous.com>
Merge pull request #37 from docker/stevvooe-patch-1
build: add 1.8 and tip to travis build
build: add 1.8 and tip to travis build
Signed-off-by: Stephen J Day <stephen.day@docker.com>
Merge pull request #36 from stevvooe/cleanup-encoding-tests
encoding: clean up unit tests
Merge pull request #30 from stevvooe/truncate-twrite-msize
channel: truncate twrite messages based on msize
encoding: clean up unit tests
Signed-off-by: Stephen J Day <stephen.day@docker.com>
channel: truncate twrite messages based on msize
While there are a few problems around handling of msize, the easiest to address and, arguably, the most problematic is that of Twrite. We now truncate Twrite.Data to the correct length if it will overflow the msize limit negotiated on the session. ErrShortWrite is returned by the `Session.Write` method if written data is truncated. In addition, we now reject incoming messages from `ReadFcall` that overflow the msize. Such messages are probably terminal in practice, but can be detected with the `Overflow` function. Tread is also handled accordingly, such that the Count field will be rewritten such that the response doesn't overflow the msize. Signed-off-by: Stephen J Day <stephen.day@docker.com>
Merge pull request #33 from stevvooe/new-context
p9p: use new context package throughout
p9p: use new context package throughout
Signed-off-by: Stephen J Day <stephen.day@docker.com>
Merge pull request #28 from simonferquel/msize_negociation_fix
Fixes the MSize comparison when negociating msize with server
Fixes the MSize comparison when negociating msize with server
Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
Merge pull request #23 from dominikh/qtype
Specify type for all constants, not just the first
Merge pull request #24 from stevvooe/refactor-transport-close
transport: remove race condition on close
Specify type for all constants, not just the first
Closes #22 Signed-off-by: Dominik Honnef <dominik@honnef.co>