Tests: rcvbuf and sndbuf directives for upstream sockets#47
Open
patrikwl wants to merge 3 commits into
Open
Conversation
Author
|
Update: I’ve extended this tests PR to cover the HTTP upstream follow-up as well. The PR now includes:
Verified locally with: ```sh Result: all tests pass. The corresponding nginx source changes are in Refs nginx/nginx#1297. |
Adds stream_proxy_socket_buf.t covering proxy_socket_rcvbuf and proxy_socket_sndbuf in the four configurable forms: "off", explicit size, "max", and default-no-directive. Each subtest asserts a small message proxies through end-to-end, exercising the setsockopt() path during outbound connect. The directives are introduced in nginx/nginx#1297.
Added proxy_socket_buf.t to cover HTTP upstream socket buffer directives, including proxy runtime checks with ss -tnpemi and configuration parsing for fastcgi, grpc, memcached, scgi, and uwsgi. Extended stream proxy socket buffer tests to verify the configured buffers on the upstream socket with ss -tnpemi.
Added tunnel_socket_rcvbuf and tunnel_socket_sndbuf parsing coverage to proxy_socket_buf.t.
11bbdbd to
53e7ec9
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds
stream_proxy_socket_buf.tcoveringproxy_socket_rcvbufand
proxy_socket_sndbufin the four configurable forms:off,explicit size,
max, and default-no-directive. Each subtestasserts a small message proxies through end-to-end, exercising
the
setsockopt()path during outbound connect.The directives are introduced in nginx/nginx#1298.
Refs nginx/nginx#1297.