@@ -1,94 +1,101 @@
1+{-# LANGUAGE BangPatterns #-}
12{-# LANGUAGE CPP #-}
23{-# LANGUAGE OverloadedStrings #-}
4+{-# LANGUAGE PackageImports #-}
5+{-# LANGUAGE PatternGuards #-}
6+{-# LANGUAGE RecordWildCards #-}
37{-# LANGUAGE ScopedTypeVariables #-}
48{-# LANGUAGE TupleSections #-}
9+{-# LANGUAGE ViewPatterns #-}
510{-# OPTIONS_GHC -fno-warn-deprecations #-}
611
712module Network.Wai.Handler.Warp.Run where
813
9−#if __GLASGOW_HASKELL__ < 709
10−import Control.Applicative ((<$>))
11−#endif
14+import "iproute" Data.IP (toHostAddress, toHostAddress6)
1215import Control.Arrow (first)
13−import Control.Concurrent (threadDelay)
1416import qualified Control.Concurrent as Conc (yield)
1517import Control.Exception as E
16−import Control.Monad (when, unless, void)
17−import Data.ByteString (ByteString)
1818import qualified Data.ByteString as S
1919import Data.Char (chr)
20−import Data.IP (toHostAddress, toHostAddress6)
2120import Data.IORef (IORef, newIORef, readIORef, writeIORef)
2221import Data.Streaming.Network (bindPortTCP)
23−import Network (sClose, Socket)
24−import Network.Socket (accept, withSocketsDo, SockAddr(SockAddrInet, SockAddrInet6))
22+import Foreign.C.Error (Errno(..), eCONNABORTED)
23+import GHC.IO.Exception (IOException(..))
24+import qualified Network.HTTP2 as H2
25+import Network.Socket (Socket, close, accept, withSocketsDo, SockAddr(SockAddrInet, SockAddrInet6), setSocketOption, SocketOption(..))
26+#if MIN_VERSION_network(3,1,1)
27+import Network.Socket (gracefulClose)
28+#endif
2529import qualified Network.Socket.ByteString as Sock
2630import Network.Wai
27−import Network.Wai.HTTP2 (HTTP2Application, promoteApplication)
31+import Network.Wai.Internal (ResponseReceived (ResponseReceived))
32+import System.Environment (lookupEnv)
33+import qualified System.TimeManager as T
34+import System.Timeout (timeout)
35+
2836import Network.Wai.Handler.Warp.Buffer
2937import Network.Wai.Handler.Warp.Counter
3038import qualified Network.Wai.Handler.Warp.Date as D
3139import qualified Network.Wai.Handler.Warp.FdCache as F
32−import Network.Wai.Handler.Warp.HTTP2 (http2, isHTTP2)
40+import qualified Network.Wai.Handler.Warp.FileInfoCache as I
41+import Network.Wai.Handler.Warp.HTTP2 (http2)
42+import Network.Wai.Handler.Warp.HTTP2.Types (isHTTP2)
3343import Network.Wai.Handler.Warp.Header
44+import Network.Wai.Handler.Warp.Imports hiding (readInt)
3445import Network.Wai.Handler.Warp.ReadInt
3546import Network.Wai.Handler.Warp.Recv
3647import Network.Wai.Handler.Warp.Request
3748import Network.Wai.Handler.Warp.Response
3849import Network.Wai.Handler.Warp.SendFile
3950import Network.Wai.Handler.Warp.Settings
40−import qualified Network.Wai.Handler.Warp.Timeout as T
4151import Network.Wai.Handler.Warp.Types
42−import Network.Wai.Internal (ResponseReceived (ResponseReceived))
43−import System.Environment (getEnvironment)
44−import System.IO.Error (isFullErrorType, ioeGetErrorType)
52+
4553
4654#if WINDOWS
4755import Network.Wai.Handler.Warp.Windows
4856#else
49−import System.Posix.IO (FdOption(CloseOnExec), setFdOption)
5057import Network.Socket (fdSocket)
5158#endif
5259
5360-- | Creating 'Connection' for plain HTTP based on a given socket.
54−socketConnection :: Socket -> IO Connection
55−socketConnection s = do
61+socketConnection :: Settings -> Socket -> IO Connection
62+#if MIN_VERSION_network(3,1,1)
63+socketConnection set s = do
64+#else
65+socketConnection _ s = do
66+#endif
5667 bufferPool <- newBufferPool
5768 writeBuf <- allocateBuffer bufferSize
5869 let sendall = Sock.sendAll s
70+ isH2 <- newIORef False -- HTTP/1.x
5971 return Connection {
6072 connSendMany = Sock.sendMany s
6173 , connSendAll = sendall
6274 , connSendFile = sendFile s writeBuf bufferSize sendall
63− , connClose = sClose s >> freeBuffer writeBuf
75+#if MIN_VERSION_network(3,1,1)
76+ , connClose = do
77+ h2 <- readIORef isH2
78+ let tm = if h2 then settingsGracefulCloseTimeout2 set
79+ else settingsGracefulCloseTimeout1 set
80+ if tm == 0 then
81+ close s
82+ else
83+ gracefulClose s tm `E.catch` \(E.SomeException _) -> return ()
84+#else
85+ , connClose = close s
86+#endif
87+ , connFree = freeBuffer writeBuf
6488 , connRecv = receive s bufferPool
6589 , connRecvBuf = receiveBuf s
6690 , connWriteBuffer = writeBuf
6791 , connBufferSize = bufferSize
92+ , connHTTP2 = isH2
6893 }
6994
70−#if __GLASGOW_HASKELL__ < 702
71−allowInterrupt :: IO ()
72−allowInterrupt = unblock $ return ()
73−#endif
74−
75−-- Composition over two arguments at once; used for runHTTP2\*.
76−(.:) :: (c -> d) -> (a -> b -> c) -> a -> b -> d
77−f .: g = curry $ f . uncurry g
78−
7995-- | Run an 'Application' on the given port.
8096-- This calls 'runSettings' with 'defaultSettings'.
8197run :: Port -> Application -> IO ()
82−run port = runServe port . serveDefault
83−
84−-- | Serve an 'HTTP2Application' and an 'Application' together on the given
85−-- port.
86−runHTTP2 :: Port -> HTTP2Application -> Application -> IO ()
87−runHTTP2 port = runServe port .: serveHTTP2
88−
89−-- | The generalized form of 'run'.
90−runServe :: Port -> ServeConnection -> IO ()
91−runServe p = runServeSettings defaultSettings { settingsPort = p }
98+run p = runSettings defaultSettings { settingsPort = p }
9299
93100-- | Run an 'Application' on the port present in the @PORT@
94101-- environment variable. Uses the 'Port' given when the variable is unset.
@@ -96,44 +103,28 @@ runServe p = runServeSettings defaultSettings { settingsPort = p }
96103--
97104-- Since 3.0.9
98105runEnv :: Port -> Application -> IO ()
99−runEnv port = runServeEnv port . serveDefault
100−
101−-- | The HTTP\/2-aware form of 'runEnv'.
102−runHTTP2Env :: Port -> HTTP2Application -> Application -> IO ()
103−runHTTP2Env port = runServeEnv port .: serveHTTP2
104−
105−-- | The generalized form of 'runEnv'.
106−runServeEnv :: Port -> ServeConnection -> IO ()
107−runServeEnv p serveConn = do
108− mp <- lookup "PORT" <$> getEnvironment
106+runEnv p app = do
107+ mp <- lookupEnv "PORT"
109108
110− maybe (runServe p serveConn) runReadPort mp
109+ maybe (run p app) runReadPort mp
111110
112111 where
113112 runReadPort :: String -> IO ()
114113 runReadPort sp = case reads sp of
115− ((p', _):_) -> runServe p' serveConn
114+ ((p', _):_) -> run p' app
116115 _ -> fail $ "Invalid value in $PORT: " ++ sp
117116
118117-- | Run an 'Application' with the given 'Settings'.
119118-- This opens a listen socket on the port defined in 'Settings' and
120119-- calls 'runSettingsSocket'.
121120runSettings :: Settings -> Application -> IO ()
122−runSettings set = runServeSettings set . serveDefault
123−
124−-- | The HTTP\/2-aware form of 'runSettings'.
125−runHTTP2Settings :: Settings -> HTTP2Application -> Application -> IO ()
126−runHTTP2Settings set = runServeSettings set .: serveHTTP2
127−
128−-- | The generalized form of 'runSettings'.
129−runServeSettings :: Settings -> ServeConnection -> IO ()
130−runServeSettings set serveConn = withSocketsDo $
121+runSettings set app = withSocketsDo $
131122 bracket
132123 (bindPortTCP (settingsPort set) (settingsHost set))
133− sClose
124+ close
134125 (\socket -> do
135126 setSocketCloseOnExec socket
136− runServeSettingsSocket set socket serveConn)
127+ runSettingsSocket set socket app)
137128
138129-- | This installs a shutdown handler for the given socket and
139130-- calls 'runSettingsConnection' with the default connection setup action
@@ -147,22 +138,9 @@ runServeSettings set serveConn = withSocketsDo $
147138-- Note that the 'settingsPort' will still be passed to 'Application's via the
148139-- 'serverPort' record.
149140runSettingsSocket :: Settings -> Socket -> Application -> IO ()
150−runSettingsSocket set socket = runServeSettingsSocket set socket . serveDefault
151−
152−-- | The HTTP\/2-aware form of 'runSettingsSocket'.
153−runHTTP2SettingsSocket :: Settings
154− -> Socket
155− -> HTTP2Application
156− -> Application
157− -> IO ()
158−runHTTP2SettingsSocket set socket =
159− runServeSettingsSocket set socket .: serveHTTP2
160−
161−-- | The generalized form of 'runSettingsSocket'.
162−runServeSettingsSocket :: Settings -> Socket -> ServeConnection -> IO ()
163−runServeSettingsSocket set socket serveConn = do
141+runSettingsSocket set socket app = do
164142 settingsInstallShutdownHandler set closeListenSocket
165− runServeSettingsConnection set getConn serveConn
143+ runSettingsConnection set getConn app
166144 where
167145 getConn = do
168146#if WINDOWS
@@ -171,10 +149,12 @@ runServeSettingsSocket set socket serveConn = do
171149 (s, sa) <- accept socket
172150#endif
173151 setSocketCloseOnExec s
174− conn <- socketConnection s
152+ -- NoDelay causes an error for AF_UNIX.
153+ setSocketOption s NoDelay 1 `E.catch` \(E.SomeException _) -> return ()
154+ conn <- socketConnection set s
175155 return (conn, sa)
176156
177− closeListenSocket = sClose socket
157+ closeListenSocket = close socket
178158
179159-- | The connection setup action would be expensive. A good example
180160-- is initialization of TLS.
@@ -186,16 +166,7 @@ runServeSettingsSocket set socket serveConn = do
186166--
187167-- Since 1.3.5
188168runSettingsConnection :: Settings -> IO (Connection, SockAddr) -> Application -> IO ()
189−runSettingsConnection set getConn =
190− runServeSettingsConnection set getConn . serveDefault
191−
192−-- | The generalized form of 'runSettingsConnection'.
193−runServeSettingsConnection :: Settings
194− -> IO (Connection, SockAddr)
195− -> ServeConnection
196− -> IO ()
197−runServeSettingsConnection set getConn serveConn =
198− runServeSettingsConnectionMaker set getConnMaker serveConn
169+runSettingsConnection set getConn app = runSettingsConnectionMaker set getConnMaker app
199170 where
200171 getConnMaker = do
201172 (conn, sa) <- getConn
@@ -204,16 +175,8 @@ runServeSettingsConnection set getConn serveConn =
204175-- | This modifies the connection maker so that it returns 'TCP' for 'Transport'
205176-- (i.e. plain HTTP) then calls 'runSettingsConnectionMakerSecure'.
206177runSettingsConnectionMaker :: Settings -> IO (IO Connection, SockAddr) -> Application -> IO ()
207−runSettingsConnectionMaker set getConnMaker =
208− runServeSettingsConnectionMaker set getConnMaker . serveDefault
209−
210−-- | The generalized form of 'runSettingsConnectionMaker'.
211−runServeSettingsConnectionMaker :: Settings
212− -> IO (IO Connection, SockAddr)
213− -> ServeConnection
214− -> IO ()
215−runServeSettingsConnectionMaker x y =
216− runServeSettingsConnectionMakerSecure x (toTCP <$> y)
178+runSettingsConnectionMaker x y =
179+ runSettingsConnectionMakerSecure x (toTCP <$> y)
217180 where
218181 toTCP = first ((, TCP) <$>)
219182
@@ -226,28 +189,26 @@ runServeSettingsConnectionMaker x y =
226189--
227190-- Since 2.1.4
228191runSettingsConnectionMakerSecure :: Settings -> IO (IO (Connection, Transport), SockAddr) -> Application -> IO ()
229−runSettingsConnectionMakerSecure set getConnMaker =
230− runServeSettingsConnectionMakerSecure set getConnMaker . serveDefault
231−
232−-- | The generalized form of 'runSettingsConnectionMakerSecure'.
233−runServeSettingsConnectionMakerSecure :: Settings
234− -> IO (IO (Connection, Transport), SockAddr)
235− -> ServeConnection
236− -> IO ()
237−runServeSettingsConnectionMakerSecure set getConnMaker serveConn = do
192+runSettingsConnectionMakerSecure set getConnMaker app = do
238193 settingsBeforeMainLoop set
239194 counter <- newCounter
240−
241− D.withDateCache $ \dc ->
242− F.withFdCache fdCacheDurationInSeconds $ \fc ->
243− withTimeoutManager $ \tm ->
244− acceptConnection set getConnMaker serveConn dc fc tm counter
195+ withII $ acceptConnection set getConnMaker app counter
245196 where
246− fdCacheDurationInSeconds = settingsFdCacheDuration set * 1000000
197+ withII action =
198+ withTimeoutManager $ \tm ->
199+ D.withDateCache $ \dc ->
200+ F.withFdCache fdCacheDurationInSeconds $ \fdc ->
201+ I.withFileInfoCache fdFileInfoDurationInSeconds $ \fic -> do
202+ let ii = InternalInfo tm dc fdc fic
203+ action ii
204+
205+ !fdCacheDurationInSeconds = settingsFdCacheDuration set * 1000000
206+ !fdFileInfoDurationInSeconds = settingsFileInfoCacheDuration set * 1000000
207+ !timeoutInSeconds = settingsTimeout set * 1000000
247208 withTimeoutManager f = case settingsManager set of
248209 Just tm -> f tm
249210 Nothing -> bracket
250− (T.initialize $ settingsTimeout set * 1000000)
211+ (T.initialize timeoutInSeconds)
251212 T.stopManager
252213 f
253214
@@ -266,18 +227,21 @@ runServeSettingsConnectionMakerSecure set getConnMaker serveConn = do
266227-- Our approach is explained in the comments below.
267228acceptConnection :: Settings
268229 -> IO (IO (Connection, Transport), SockAddr)
269− -> ServeConnection
270− -> D.DateCache
271− -> Maybe F.MutableFdCache
272− -> T.Manager
230+ -> Application
273231 -> Counter
232+ -> InternalInfo
274233 -> IO ()
275−acceptConnection set getConnMaker serveConn dc fc tm counter = do
234+acceptConnection set getConnMaker app counter ii = do
276235 -- First mask all exceptions in acceptLoop. This is necessary to
277236 -- ensure that no async exception is throw between the call to
278237 -- acceptNewConnection and the registering of connClose.
238+ --
239+ -- acceptLoop can be broken by closing the listing socket.
279240 void $ mask_ acceptLoop
280− gracefulShutdown counter
241+ -- In some cases, we want to stop Warp here without graceful shutdown.
242+ -- So, async exceptions are allowed here.
243+ -- That's why `finally` is not used.
244+ gracefulShutdown set counter
281245 where
282246 acceptLoop = do
283247 -- Allow async exceptions before receiving the next connection maker.
@@ -294,90 +258,78 @@ acceptConnection set getConnMaker serveConn dc fc tm counter = do
294258 case mx of
295259 Nothing -> return ()
296260 Just (mkConn, addr) -> do
297− fork set mkConn addr serveConn dc fc tm counter
261+ fork set mkConn addr app counter ii
298262 acceptLoop
299263
300264 acceptNewConnection = do
301265 ex <- try getConnMaker
302266 case ex of
303267 Right x -> return $ Just x
304− Left e -> do
305− settingsOnException set Nothing $ toException e
306− if isFullErrorType (ioeGetErrorType e) then do
307− -- "resource exhausted (Too many open files)" may
308− -- happen by accept(). Wait a second hoping that
309− -- resource will be available.
310− threadDelay 1000000
311− acceptNewConnection
312− else
313− -- Assuming the listen socket is closed.
314− return Nothing
268+ Left e -> do
269+ let eConnAborted = getErrno eCONNABORTED
270+ getErrno (Errno cInt) = cInt
271+ if ioe_errno e == Just eConnAborted
272+ then acceptNewConnection
273+ else do
274+ settingsOnException set Nothing $ toException e
275+ return Nothing
315276
316277-- Fork a new worker thread for this connection maker, and ask for a
317278-- function to unmask (i.e., allow async exceptions to be thrown).
318279fork :: Settings
319280 -> IO (Connection, Transport)
320281 -> SockAddr
321− -> ServeConnection
322− -> D.DateCache
323− -> Maybe F.MutableFdCache
324− -> T.Manager
282+ -> Application
325283 -> Counter
284+ -> InternalInfo
326285 -> IO ()
327−fork set mkConn addr serveConn dc fc tm counter = settingsFork set $ \ unmask ->
328− -- Run the connection maker to get a new connection, and ensure
329− -- that the connection is closed. If the mkConn call throws an
330− -- exception, we will leak the connection. If the mkConn call is
331− -- vulnerable to attacks (e.g., Slowloris), we do nothing to
332− -- protect the server. It is therefore vital that mkConn is well
333− -- vetted.
334− --
335− -- We grab the connection before registering timeouts since the
336− -- timeouts will be useless during connection creation, due to the
337− -- fact that async exceptions are still masked.
338− bracket mkConn closeConn $ \(conn, transport) ->
286+fork set mkConn addr app counter ii = settingsFork set $ \unmask ->
287+ -- Call the user-supplied on exception code if any
288+ -- exceptions are thrown.
289+ handle (settingsOnException set Nothing) $
290+ -- Run the connection maker to get a new connection, and ensure
291+ -- that the connection is closed. If the mkConn call throws an
292+ -- exception, we will leak the connection. If the mkConn call is
293+ -- vulnerable to attacks (e.g., Slowloris), we do nothing to
294+ -- protect the server. It is therefore vital that mkConn is well
295+ -- vetted.
296+ --
297+ -- We grab the connection before registering timeouts since the
298+ -- timeouts will be useless during connection creation, due to the
299+ -- fact that async exceptions are still masked.
300+ bracket mkConn cleanUp (serve unmask)
301+ where
302+ cleanUp (conn, _) = connClose conn `finally` connFree conn
339303
340304 -- We need to register a timeout handler for this thread, and
341305 -- cancel that handler as soon as we exit.
342− bracket (T.registerKillThread tm) T.cancel $ \th ->
343−
344− let ii = InternalInfo th tm fc dc
345− -- We now have fully registered a connection close handler
346− -- in the case of all exceptions, so it is safe to one
347− -- again allow async exceptions.
348− in unmask .
349− -- Call the user-supplied on exception code if any
350− -- exceptions are thrown.
351− handle (settingsOnException set Nothing) .
352−
353− -- Call the user-supplied code for connection open and close events
354− bracket (onOpen addr) (onClose addr) $ \goingon ->
355−
356− -- Actually serve this connection.
357− -- bracket with closeConn above ensures the connection is closed.
358− when goingon $ serveConn conn ii addr transport set
359− where
360− closeConn (conn, _transport) = connClose conn
306+ serve unmask (conn, transport) = bracket register cancel $ \th -> do
307+ -- We now have fully registered a connection close handler in
308+ -- the case of all exceptions, so it is safe to once again
309+ -- allow async exceptions.
310+ unmask .
311+ -- Call the user-supplied code for connection open and
312+ -- close events
313+ bracket (onOpen addr) (onClose addr) $ \goingon ->
314+ -- Actually serve this connection. bracket with closeConn
315+ -- above ensures the connection is closed.
316+ when goingon $ serveConnection conn ii th addr transport set app
317+ where
318+ register = T.registerKillThread (timeoutManager ii) (connClose conn)
319+ cancel = T.cancel
361320
362321 onOpen adr = increase counter >> settingsOnOpen set adr
363322 onClose adr _ = decrease counter >> settingsOnClose set adr
364323
365−-- The type of a function to serve a fully-prepared connection.
366−type ServeConnection = Connection
367− -> InternalInfo
368− -> SockAddr
369− -> Transport
370− -> Settings
371− -> IO ()
372−
373−-- Serve an HTTP\/2-unaware Application to a connection over any HTTP version.
374−serveDefault :: Application -> ServeConnection
375−serveDefault app = serveHTTP2 (promoteApplication app) app
376−
377−-- Serve an HTTP\/2-aware application over HTTP\/2 or a backup 'Application'
378−-- over HTTP\/1.1 or HTTP\/1.
379−serveHTTP2 :: HTTP2Application -> Application -> ServeConnection
380−serveHTTP2 app2 app conn ii origAddr transport settings = do
324+serveConnection :: Connection
325+ -> InternalInfo
326+ -> T.Handle
327+ -> SockAddr
328+ -> Transport
329+ -> Settings
330+ -> Application
331+ -> IO ()
332+serveConnection conn ii th origAddr transport settings app = do
381333 -- fixme: Upgrading to HTTP/2 should be supported.
382334 (h2,bs) <- if isHTTP2 transport then
383335 return (True, "")
@@ -387,19 +339,39 @@ serveHTTP2 app2 app conn ii origAddr transport settings = do
387339 return (True, bs0)
388340 else
389341 return (False, bs0)
342+ istatus <- newIORef False
390343 if settingsHTTP2Enabled settings && h2 then do
391− recvN <- makeReceiveN bs (connRecv conn) (connRecvBuf conn)
344+ rawRecvN <- makeReceiveN bs (connRecv conn) (connRecvBuf conn)
345+ -- This thread becomes the sender in http2 library.
346+ -- In the case of event source, one request comes and one
347+ -- worker gets busy. But it is likely that the receiver does
348+ -- not receive any data at all while the sender is sending
349+ -- output data from the worker. It's not good enough to tickle
350+ -- the time handler in the receiver only. So, we should tickle
351+ -- the time handler in both the receiver and the sender.
352+ let recvN = wrappedRecvN th istatus (settingsSlowlorisSize settings) rawRecvN
353+ sendBS x = connSendAll conn x >> T.tickle th
392354 -- fixme: origAddr
393− http2 conn ii origAddr transport settings recvN app2
355+ checkTLS
356+ setConnHTTP2 conn True
357+ http2 conn transport ii origAddr settings recvN sendBS app
394358 else do
395− istatus <- newIORef False
396359 src <- mkSource (wrappedRecv conn th istatus (settingsSlowlorisSize settings))
397360 writeIORef istatus True
398361 leftoverSource src bs
399362 addr <- getProxyProtocolAddr src
400− http1 addr istatus src `E.catch` \e -> do
401− sendErrorResponse addr istatus e
402− throwIO (e :: SomeException)
363+ http1 True addr istatus src `E.catch` \e ->
364+ case () of
365+ ()
366+ -- See comment below referencing
367+ -- https://github.com/yesodweb/wai/issues/618
368+ | Just NoKeepAliveRequest <- fromException e -> return ()
369+ -- No valid request
370+ | Just (BadFirstLine _) <- fromException e -> return ()
371+ | otherwise -> do
372+ _ <- sendErrorResponse (dummyreq addr) istatus e
373+ throwIO e
374+
403375 where
404376 getProxyProtocolAddr src =
405377 case settingsProxyProtocol settings of
@@ -441,30 +413,38 @@ serveHTTP2 app2 app conn ii origAddr transport settings = do
441413
442414 decodeAscii = map (chr . fromEnum) . S.unpack
443415
444− th = threadHandle ii
416+ shouldSendErrorResponse se
417+ | Just ConnectionClosedByPeer <- fromException se = False
418+ | otherwise = True
445419
446− sendErrorResponse addr istatus e = do
420+ sendErrorResponse req istatus e = do
447421 status <- readIORef istatus
448− when status $ void $
449− sendResponse
450− (settingsServerName settings)
451− conn ii (dummyreq addr) defaultIndexRequestHeader (return S.empty) (errorResponse e)
422+ if shouldSendErrorResponse e && status
423+ then do
424+ sendResponse settings conn ii th req defaultIndexRequestHeader (return S.empty) (errorResponse e)
425+ else return False
452426
453427 dummyreq addr = defaultRequest { remoteHost = addr }
454428
455429 errorResponse e = settingsOnExceptionResponse settings e
456430
457− http1 addr istatus src = do
458− (req', mremainingRef, idxhdr, nextBodyFlush) <- recvRequest settings conn ii addr src
459− let req = req' { isSecure = isTransportSecure transport }
431+ http1 firstRequest addr istatus src = do
432+ (req, mremainingRef, idxhdr, nextBodyFlush) <- recvRequest firstRequest settings conn ii th addr src transport
460433 keepAlive <- processRequest istatus src req mremainingRef idxhdr nextBodyFlush
461434 `E.catch` \e -> do
462− -- Call the user-supplied exception handlers, passing the request.
463− sendErrorResponse addr istatus e
464435 settingsOnException settings (Just req) e
465436 -- Don't throw the error again to prevent calling settingsOnException twice.
466437 return False
467− when keepAlive $ http1 addr istatus src
438+
439+ -- When doing a keep-alive connection, the other side may just
440+ -- close the connection. We don't want to treat that as an
441+ -- exceptional situation, so we pass in False to http1 (which
442+ -- in turn passes in False to recvRequest), indicating that
443+ -- this is not the first request. If, when trying to read the
444+ -- request headers, no data is available, recvRequest will
445+ -- throw a NoKeepAliveRequest exception, which we catch here
446+ -- and ignore. See: https://github.com/yesodweb/wai/issues/618
447+ when keepAlive $ http1 False addr istatus src
468448
469449 processRequest istatus src req mremainingRef idxhdr nextBodyFlush = do
470450 -- Let the application run for as long as it wants
@@ -474,32 +454,38 @@ serveHTTP2 app2 app conn ii origAddr transport settings = do
474454 -- creating the request, we need to make sure that we don't get
475455 -- an async exception before calling the ResponseSource.
476456 keepAliveRef <- newIORef $ error "keepAliveRef not filled"
477− _ <- app req $ \res -> do
457+ r <- E.try $ app req $ \res -> do
478458 T.resume th
479459 -- FIXME consider forcing evaluation of the res here to
480460 -- send more meaningful error messages to the user.
481461 -- However, it may affect performance.
482462 writeIORef istatus False
483− keepAlive <- sendResponse
484− (settingsServerName settings)
485− conn ii req idxhdr (readSource src) res
463+ keepAlive <- sendResponse settings conn ii th req idxhdr (readSource src) res
486464 writeIORef keepAliveRef keepAlive
487465 return ResponseReceived
466+ case r of
467+ Right ResponseReceived -> return ()
468+ Left e@(SomeException _)
469+ | Just (ExceptionInsideResponseBody e') <- fromException e -> throwIO e'
470+ | otherwise -> do
471+ keepAlive <- sendErrorResponse req istatus e
472+ settingsOnException settings (Just req) e
473+ writeIORef keepAliveRef keepAlive
474+
488475 keepAlive <- readIORef keepAliveRef
489476
490477 -- We just send a Response and it takes a time to
491478 -- receive a Request again. If we immediately call recv,
492− -- it is likely to fail and the IO manager works.
493− -- It is very costly. So, we yield to another Haskell
479+ -- it is likely to fail and cause the IO manager to do some work.
480+ -- It is very costly, so we yield to another Haskell
494481 -- thread hoping that the next Request will arrive
495482 -- when this Haskell thread will be re-scheduled.
496483 -- This improves performance at least when
497484 -- the number of cores is small.
498485 Conc.yield
499486
500− if not keepAlive then
501− return False
502− else
487+ if keepAlive
488+ then
503489 -- If there is an unknown or large amount of data to still be read
504490 -- from the request body, simple drop this connection instead of
505491 -- reading it all in to satisfy a keep-alive request.
@@ -525,6 +511,21 @@ serveHTTP2 app2 app conn ii origAddr transport settings = do
525511 else
526512 return False
527513 Nothing -> tryKeepAlive
514+ else
515+ return False
516+
517+ checkTLS = case transport of
518+ TCP -> return () -- direct
519+ tls -> unless (tls12orLater tls) $ goaway conn H2.InadequateSecurity "Weak TLS"
520+ tls12orLater tls = tlsMajorVersion tls == 3 && tlsMinorVersion tls >= 3
521+
522+-- connClose must not be called here since Run:fork calls it
523+goaway :: Connection -> H2.ErrorCodeId -> ByteString -> IO ()
524+goaway Connection{..} etype debugmsg = connSendAll bytestream
525+ where
526+ einfo = H2.encodeInfo id 0
527+ frame = H2.GoAwayFrame 0 etype debugmsg
528+ bytestream = H2.encodeFrame einfo frame
528529
529530flushEntireBody :: IO ByteString -> IO ()
530531flushEntireBody src =
@@ -557,14 +558,42 @@ wrappedRecv Connection { connRecv = recv } th istatus slowlorisSize = do
557558 when (S.length bs >= slowlorisSize) $ T.tickle th
558559 return bs
559560
561+wrappedRecvN :: T.Handle -> IORef Bool -> Int -> (BufSize -> IO ByteString) -> (BufSize -> IO ByteString)
562+wrappedRecvN th istatus slowlorisSize readN bufsize = do
563+ bs <- readN bufsize
564+ unless (S.null bs) $ do
565+ writeIORef istatus True
566+ -- TODO: think about the slowloris protection in HTTP2: current code
567+ -- might open a slow-loris attack vector. Rather than timing we should
568+ -- consider limiting the per-client connections assuming that in HTTP2
569+ -- we should allow only few connections per host (real-world
570+ -- deployments with large NATs may be trickier).
571+ when (S.length bs >= slowlorisSize || bufsize <= slowlorisSize) $ T.tickle th
572+ return bs
573+
574+-- | Set flag FileCloseOnExec flag on a socket (on Unix)
575+--
560576-- Copied from: https://github.com/mzero/plush/blob/master/src/Plush/Server/Warp.hs
577+--
578+-- @since 3.2.17
561579setSocketCloseOnExec :: Socket -> IO ()
562580#if WINDOWS
563581setSocketCloseOnExec _ = return ()
564582#else
565−setSocketCloseOnExec socket =
566− setFdOption (fromIntegral $ fdSocket socket) CloseOnExec True
583+setSocketCloseOnExec socket = do
584+#if MIN_VERSION_network(3,0,0)
585+ fd <- fdSocket socket
586+#else
587+ let fd = fdSocket socket
588+#endif
589+ F.setFileCloseOnExec $ fromIntegral fd
567590#endif
568591
569−gracefulShutdown :: Counter -> IO ()
570−gracefulShutdown counter = waitForZero counter
592+gracefulShutdown :: Settings -> Counter -> IO ()
593+gracefulShutdown set counter =
594+ case settingsGracefulShutdownTimeout set of
595+ Nothing ->
596+ waitForZero counter
597+ (Just seconds) ->
598+ void (timeout (seconds * microsPerSecond) (waitForZero counter))
599+ where microsPerSecond = 1000000