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
Improvement: refine CLI commands, arguments, and communication with server documentation #681
Conversation
@Isaac-DeFrain, I know some tests are failing, but I am not sure where I caused the issues. It looks like on this one there was an unexpected end reading from the socket due to additional bytes. |
67fe410
to
e34359f
Compare
Turns out the buffer size (via |
I like all the structural changes and it simplifies our manual unrolling of the payload. I have some subjective comments that doesn't affect the validity of this PR. |
Implements #650.
Many of these commits could (should?) be squashed. I have left them for the moment for ease of review (commit-by-commit).