Skip to content

Commit

Permalink
Clarify what request-header-size and response-header-size mean
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
mikekap committed May 4, 2021
1 parent 99ece0b commit 0a740f1
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions ring-jetty-adapter/src/ring/adapter/jetty.clj
Original file line number Diff line number Diff line change
Expand Up @@ -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))]
Expand Down

0 comments on commit 0a740f1

Please sign in to comment.