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
MPTCP: implementation of mptcp #1
Conversation
this is a test implementatation of mptcp fot the http context of the nginx server
src/core/ngx_connection.c
Outdated
@@ -487,8 +488,12 @@ ngx_open_listening_sockets(ngx_cycle_t *cycle) | |||
continue; | |||
} | |||
|
|||
#if (NGX_HAVE_MPTCP) | |||
s = ngx_socket(ls[i].sockaddr->sa_family, ls[i].type, | |||
ls[i].protocol); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alignment not ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it ok now? not quite sure
this is a test implementatation of mptcp fot the http context of the nginx server fix: replace socket call swap w/ fallback fix: alignment, hopefully fix: removed compilator mptcp test add: an option mptcp for streams fix: spacing fix: removed useless pre-processors fix: value already zero
my bad, branch is back |
I guess you need to re-open the PR, or create a new one. |
still not proficient with git. i created a new PR |
This PR is internal, and aims to implement MPTCP (defined in RFC8684) into the nginx socket creation process.
The current approach is to add an option to the
listen
directive (namelymptcp
) to enable and disable the protocol.It would be interesting to have it set by the
http
orserver
blocks but I don't currently see a good way of doing so.In this initial commit MPTCP is disabled by default simply because it is easy to implement, but it is preferable to have is on by default and give the option to user to explicitly disable it.