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.

Sign up for GitHub

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

Return HTTP response to RX_OVERFLOW #4207

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Return HTTP response to RX_OVERFLOW #4207

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
daf82e4
params: Add req_overflow_status parameter
cartoush Jul 29, 2024
66c7e0f
http1_fsm: Allow HTTP response on req param overflow
cartoush Jul 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
params: Add req_overflow_status parameter 
Adds a Varnish parameter for whether and which
HTTP response code should be sent in case of a
request parameter overflow.
The default value (500) keeps the old behavior
which silently closes the connection.
  • Loading branch information
@cartoush @nigoroll
cartoush authored and nigoroll committed Jan 22, 2025
commit daf82e4fd913e85f92fecd0d205194eb9f9e202d
12 changes: 12 additions & 0 deletions include/tbl/params.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -869,6 +869,18 @@ PARAM_SIMPLE(
"IPv4 and IPv6 addresses."
)

PARAM_SIMPLE(
/* name */ req_overflow_status,
/* type */ uint,
/* min */ "400",
/* max */ "500",
/* def */ "500",
/* units */ "HTTP code",
/* descr */
"HTTP code to be returned in case of a request overflow."
"Default value 500 closes connection silently without sending HTTP response."
)

PARAM_SIMPLE(
/* name */ rush_exponent,
/* type */ uint,
Expand Down