Commit Briefs
Remove NOTAG handling from allocateTag
Signed-off-by: Kenny Levinsen <w@kl.wtf>
Provide hint for tag allocation routine
Signed-off-by: Kenny Levinsen <w@kl.wtf>
Remove tag defaulting in newFcall
Defaulting the tag in newFcall might result in unexpected behaviour. For example, if a client (erroneously) sends a Tversion on a tag, the server response will not be on a tag. newFcall should just create the struct. Signed-off-by: Kenny Levinsen <w@kl.wtf>
Rework transport tag allocation logic
New tag allocation system that avoids tag collisions and incorrect use of NOTAG. Signed-off-by: Kenny Levinsen <w@kl.wtf>
Merge pull request #13 from stevvooe/document-exported-functions
lint/vet: address several issues identified by lint/vet
lint/vet: address several issues identified by lint/vet
Signed-off-by: Stephen J Day <stephen.day@docker.com>
Merge pull request #12 from docker/move-email
Moved to official docker email
Moved to official docker email
Signed-off-by: French Ben <frenchben@docker.com>
Merge pull request #11 from thaJeztah/add-maintainers-file
Add MAINTAINERS and Contributing.md
Add MAINTAINERS and Contributing.md
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Merge pull request #9 from stevvooe/more-badges
badges: more badges for great health
Merge pull request #10 from stevvooe/log-unknown-client-tag
client: log messages with unknown tags
badges: more badges for great health
Adding the Go report card badge. Signed-off-by: Stephen J Day <stephen.day@docker.com>
client: log messages with unknown tags
Several clients have experienced panics after receiving unknown tags. The exact cause of these unknown messages is unknown. It is also not clear whether an unknown tag should be fatal to the session. This change provides more information in the panic message which we can then use to better understand this condition. Signed-off-by: Stephen J Day <stephen.day@docker.com>
Merge pull request #8 from stevvooe/err-instead-nil
errors: return errors rather than nil