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.

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

Send version along with handshake response #103

Merged
merged 4 commits into from
Jan 21, 2015
Merged

Conversation

emilsoman
Copy link
Contributor

No description provided.

@gnufied
Copy link
Contributor

gnufied commented Jan 21, 2015

I think, we should internally have protocol version, rather than reuse Rbkit version for this. That way, Rbkit version can be independently changed as long as protocol version remains the same.

@gnufied
Copy link
Contributor

gnufied commented Jan 21, 2015

Lets think of it like, no matter which nginx version I am running as long as it can speak http1.1 the browser can display the content.

gnufied pushed a commit that referenced this pull request Jan 21, 2015
Send version along with handshake response
@gnufied gnufied merged commit 4a923a7 into master Jan 21, 2015
@gnufied gnufied deleted the version-handshake branch January 21, 2015 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants