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

Don't send endless newlines on EOF #1

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@bartbes

bartbes commented Jul 16, 2015

The title is pretty self-explanatory. Since the spec says the connection should be closed by the client, I chose to not close the connection in case of EOF.

To verify this behaviour, press Ctrl-D (EOT) when using netcat.

@roryokane

This comment has been minimized.

Show comment
Hide comment
@roryokane

roryokane Jul 16, 2015

I was just about to try fixing this. Glad to see someone else already did.

roryokane commented Jul 16, 2015

I was just about to try fixing this. Glad to see someone else already did.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment