From 441c3622f4afef830013460dcb1f54ffe2884590 Mon Sep 17 00:00:00 2001 From: Mike Kaplinskiy Date: Mon, 3 May 2021 17:15:27 -0700 Subject: [PATCH] Clarify what {request,response}-header-size mean Jetty's config options are for the PROTOCOL header, not a single Field: Value pair. So these options control the sum of all headers and status lines. --- ring-jetty-adapter/src/ring/adapter/jetty.clj | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/ring-jetty-adapter/src/ring/adapter/jetty.clj b/ring-jetty-adapter/src/ring/adapter/jetty.clj index 143cba70..ae9de88d 100644 --- a/ring-jetty-adapter/src/ring/adapter/jetty.clj +++ b/ring-jetty-adapter/src/ring/adapter/jetty.clj @@ -203,8 +203,10 @@ :need,:want or :none (defaults to :none) :send-date-header? - add a date header to the response (default true) :output-buffer-size - the response body buffer size (default 32768) - :request-header-size - the maximum size of a request header (default 8192) - :response-header-size - the maximum size of a response header (default 8192) + :request-header-size - the maximum size of the request line and headers + (i.e. the request protocol header) (default 8192) + :response-header-size - the maximum size of the response line and headers + (i.e. the response protocol header) (default 8192) :send-server-version? - add Server header to HTTP response (default true)" [handler options] (let [server (create-server (dissoc options :configurator))]