@@ -1,95 +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
3240import qualified Network.Wai.Handler.Warp.FileInfoCache as I
33−import Network.Wai.Handler.Warp.HTTP2 (http2, isHTTP2)
41+import Network.Wai.Handler.Warp.HTTP2 (http2)
42+import Network.Wai.Handler.Warp.HTTP2.Types (isHTTP2)
3443import Network.Wai.Handler.Warp.Header
44+import Network.Wai.Handler.Warp.Imports hiding (readInt)
3545import Network.Wai.Handler.Warp.ReadInt
3646import Network.Wai.Handler.Warp.Recv
3747import Network.Wai.Handler.Warp.Request
3848import Network.Wai.Handler.Warp.Response
3949import Network.Wai.Handler.Warp.SendFile
4050import Network.Wai.Handler.Warp.Settings
41−import qualified Network.Wai.Handler.Warp.Timeout as T
4251import Network.Wai.Handler.Warp.Types
43−import Network.Wai.Internal (ResponseReceived (ResponseReceived))
44−import System.Environment (getEnvironment)
45−import System.IO.Error (isFullErrorType, ioeGetErrorType)
52+
4653
4754#if WINDOWS
4855import Network.Wai.Handler.Warp.Windows
4956#else
50−import System.Posix.IO (FdOption(CloseOnExec), setFdOption)
5157import Network.Socket (fdSocket)
5258#endif
5359
5460-- | Creating 'Connection' for plain HTTP based on a given socket.
55−socketConnection :: Socket -> IO Connection
56−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
5767 bufferPool <- newBufferPool
5868 writeBuf <- allocateBuffer bufferSize
5969 let sendall = Sock.sendAll s
70+ isH2 <- newIORef False -- HTTP/1.x
6071 return Connection {
6172 connSendMany = Sock.sendMany s
6273 , connSendAll = sendall
6374 , connSendFile = sendFile s writeBuf bufferSize sendall
64− , 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
6588 , connRecv = receive s bufferPool
6689 , connRecvBuf = receiveBuf s
6790 , connWriteBuffer = writeBuf
6891 , connBufferSize = bufferSize
92+ , connHTTP2 = isH2
6993 }
7094
71−#if __GLASGOW_HASKELL__ < 702
72−allowInterrupt :: IO ()
73−allowInterrupt = unblock $ return ()
74−#endif
75−
76−-- Composition over two arguments at once; used for runHTTP2\*.
77−(.:) :: (c -> d) -> (a -> b -> c) -> a -> b -> d
78−f .: g = curry $ f . uncurry g
79−
8095-- | Run an 'Application' on the given port.
8196-- This calls 'runSettings' with 'defaultSettings'.
8297run :: Port -> Application -> IO ()
83−run port = runServe port . serveDefault
84−
85−-- | Serve an 'HTTP2Application' and an 'Application' together on the given
86−-- port.
87−runHTTP2 :: Port -> HTTP2Application -> Application -> IO ()
88−runHTTP2 port = runServe port .: serveHTTP2
89−
90−-- | The generalized form of 'run'.
91−runServe :: Port -> ServeConnection -> IO ()
92−runServe p = runServeSettings defaultSettings { settingsPort = p }
98+run p = runSettings defaultSettings { settingsPort = p }
9399
94100-- | Run an 'Application' on the port present in the @PORT@
95101-- environment variable. Uses the 'Port' given when the variable is unset.
@@ -97,44 +103,28 @@ runServe p = runServeSettings defaultSettings { settingsPort = p }
97103--
98104-- Since 3.0.9
99105runEnv :: Port -> Application -> IO ()
100−runEnv port = runServeEnv port . serveDefault
101−
102−-- | The HTTP\/2-aware form of 'runEnv'.
103−runHTTP2Env :: Port -> HTTP2Application -> Application -> IO ()
104−runHTTP2Env port = runServeEnv port .: serveHTTP2
105−
106−-- | The generalized form of 'runEnv'.
107−runServeEnv :: Port -> ServeConnection -> IO ()
108−runServeEnv p serveConn = do
109− mp <- lookup "PORT" <$> getEnvironment
106+runEnv p app = do
107+ mp <- lookupEnv "PORT"
110108
111− maybe (runServe p serveConn) runReadPort mp
109+ maybe (run p app) runReadPort mp
112110
113111 where
114112 runReadPort :: String -> IO ()
115113 runReadPort sp = case reads sp of
116− ((p', _):_) -> runServe p' serveConn
114+ ((p', _):_) -> run p' app
117115 _ -> fail $ "Invalid value in $PORT: " ++ sp
118116
119117-- | Run an 'Application' with the given 'Settings'.
120118-- This opens a listen socket on the port defined in 'Settings' and
121119-- calls 'runSettingsSocket'.
122120runSettings :: Settings -> Application -> IO ()
123−runSettings set = runServeSettings set . serveDefault
124−
125−-- | The HTTP\/2-aware form of 'runSettings'.
126−runHTTP2Settings :: Settings -> HTTP2Application -> Application -> IO ()
127−runHTTP2Settings set = runServeSettings set .: serveHTTP2
128−
129−-- | The generalized form of 'runSettings'.
130−runServeSettings :: Settings -> ServeConnection -> IO ()
131−runServeSettings set serveConn = withSocketsDo $
121+runSettings set app = withSocketsDo $
132122 bracket
133123 (bindPortTCP (settingsPort set) (settingsHost set))
134− sClose
124+ close
135125 (\socket -> do
136126 setSocketCloseOnExec socket
137− runServeSettingsSocket set socket serveConn)
127+ runSettingsSocket set socket app)
138128
139129-- | This installs a shutdown handler for the given socket and
140130-- calls 'runSettingsConnection' with the default connection setup action
@@ -148,22 +138,9 @@ runServeSettings set serveConn = withSocketsDo $
148138-- Note that the 'settingsPort' will still be passed to 'Application's via the
149139-- 'serverPort' record.
150140runSettingsSocket :: Settings -> Socket -> Application -> IO ()
151−runSettingsSocket set socket = runServeSettingsSocket set socket . serveDefault
152−
153−-- | The HTTP\/2-aware form of 'runSettingsSocket'.
154−runHTTP2SettingsSocket :: Settings
155− -> Socket
156− -> HTTP2Application
157− -> Application
158− -> IO ()
159−runHTTP2SettingsSocket set socket =
160− runServeSettingsSocket set socket .: serveHTTP2
161−
162−-- | The generalized form of 'runSettingsSocket'.
163−runServeSettingsSocket :: Settings -> Socket -> ServeConnection -> IO ()
164−runServeSettingsSocket set socket serveConn = do
141+runSettingsSocket set socket app = do
165142 settingsInstallShutdownHandler set closeListenSocket
166− runServeSettingsConnection set getConn serveConn
143+ runSettingsConnection set getConn app
167144 where
168145 getConn = do
169146#if WINDOWS
@@ -172,10 +149,12 @@ runServeSettingsSocket set socket serveConn = do
172149 (s, sa) <- accept socket
173150#endif
174151 setSocketCloseOnExec s
175− 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
176155 return (conn, sa)
177156
178− closeListenSocket = sClose socket
157+ closeListenSocket = close socket
179158
180159-- | The connection setup action would be expensive. A good example
181160-- is initialization of TLS.
@@ -187,16 +166,7 @@ runServeSettingsSocket set socket serveConn = do
187166--
188167-- Since 1.3.5
189168runSettingsConnection :: Settings -> IO (Connection, SockAddr) -> Application -> IO ()
190−runSettingsConnection set getConn =
191− runServeSettingsConnection set getConn . serveDefault
192−
193−-- | The generalized form of 'runSettingsConnection'.
194−runServeSettingsConnection :: Settings
195− -> IO (Connection, SockAddr)
196− -> ServeConnection
197− -> IO ()
198−runServeSettingsConnection set getConn serveConn =
199− runServeSettingsConnectionMaker set getConnMaker serveConn
169+runSettingsConnection set getConn app = runSettingsConnectionMaker set getConnMaker app
200170 where
201171 getConnMaker = do
202172 (conn, sa) <- getConn
@@ -205,16 +175,8 @@ runServeSettingsConnection set getConn serveConn =
205175-- | This modifies the connection maker so that it returns 'TCP' for 'Transport'
206176-- (i.e. plain HTTP) then calls 'runSettingsConnectionMakerSecure'.
207177runSettingsConnectionMaker :: Settings -> IO (IO Connection, SockAddr) -> Application -> IO ()
208−runSettingsConnectionMaker set getConnMaker =
209− runServeSettingsConnectionMaker set getConnMaker . serveDefault
210−
211−-- | The generalized form of 'runSettingsConnectionMaker'.
212−runServeSettingsConnectionMaker :: Settings
213− -> IO (IO Connection, SockAddr)
214− -> ServeConnection
215− -> IO ()
216−runServeSettingsConnectionMaker x y =
217− runServeSettingsConnectionMakerSecure x (toTCP <$> y)
178+runSettingsConnectionMaker x y =
179+ runSettingsConnectionMakerSecure x (toTCP <$> y)
218180 where
219181 toTCP = first ((, TCP) <$>)
220182
@@ -227,32 +189,26 @@ runServeSettingsConnectionMaker x y =
227189--
228190-- Since 2.1.4
229191runSettingsConnectionMakerSecure :: Settings -> IO (IO (Connection, Transport), SockAddr) -> Application -> IO ()
230−runSettingsConnectionMakerSecure set getConnMaker =
231− runServeSettingsConnectionMakerSecure set getConnMaker . serveDefault
232−
233−-- | The generalized form of 'runSettingsConnectionMakerSecure'.
234−runServeSettingsConnectionMakerSecure :: Settings
235− -> IO (IO (Connection, Transport), SockAddr)
236− -> ServeConnection
237− -> IO ()
238−runServeSettingsConnectionMakerSecure set getConnMaker serveConn = do
192+runSettingsConnectionMakerSecure set getConnMaker app = do
239193 settingsBeforeMainLoop set
240194 counter <- newCounter
241− withII $ acceptConnection set getConnMaker serveConn counter
195+ withII $ acceptConnection set getConnMaker app counter
242196 where
243197 withII action =
198+ withTimeoutManager $ \tm ->
244199 D.withDateCache $ \dc ->
245− F.withFdCache fdCacheDurationInSeconds $ \fc ->
246− I.withFileInfoCache 10 $ \get -> -- fixme: hard-coding
247− withTimeoutManager $ \tm -> do
248− let ii0 = InternalInfo undefined tm fc get dc -- fixme: undefined
249− action ii0
250−
251− fdCacheDurationInSeconds = settingsFdCacheDuration set * 1000000
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
252208 withTimeoutManager f = case settingsManager set of
253209 Just tm -> f tm
254210 Nothing -> bracket
255− (T.initialize $ settingsTimeout set * 1000000)
211+ (T.initialize timeoutInSeconds)
256212 T.stopManager
257213 f
258214
@@ -271,16 +227,21 @@ runServeSettingsConnectionMakerSecure set getConnMaker serveConn = do
271227-- Our approach is explained in the comments below.
272228acceptConnection :: Settings
273229 -> IO (IO (Connection, Transport), SockAddr)
274− -> ServeConnection
230+ -> Application
275231 -> Counter
276232 -> InternalInfo
277233 -> IO ()
278−acceptConnection set getConnMaker serveConn counter ii0 = do
234+acceptConnection set getConnMaker app counter ii = do
279235 -- First mask all exceptions in acceptLoop. This is necessary to
280236 -- ensure that no async exception is throw between the call to
281237 -- acceptNewConnection and the registering of connClose.
238+ --
239+ -- acceptLoop can be broken by closing the listing socket.
282240 void $ mask_ acceptLoop
283− 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
284245 where
285246 acceptLoop = do
286247 -- Allow async exceptions before receiving the next connection maker.
@@ -297,88 +258,78 @@ acceptConnection set getConnMaker serveConn counter ii0 = do
297258 case mx of
298259 Nothing -> return ()
299260 Just (mkConn, addr) -> do
300− fork set mkConn addr serveConn counter ii0
261+ fork set mkConn addr app counter ii
301262 acceptLoop
302263
303264 acceptNewConnection = do
304265 ex <- try getConnMaker
305266 case ex of
306267 Right x -> return $ Just x
307− Left e -> do
308− settingsOnException set Nothing $ toException e
309− if isFullErrorType (ioeGetErrorType e) then do
310− -- "resource exhausted (Too many open files)" may
311− -- happen by accept(). Wait a second hoping that
312− -- resource will be available.
313− threadDelay 1000000
314− acceptNewConnection
315− else
316− -- Assuming the listen socket is closed.
317− 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
318276
319277-- Fork a new worker thread for this connection maker, and ask for a
320278-- function to unmask (i.e., allow async exceptions to be thrown).
321279fork :: Settings
322280 -> IO (Connection, Transport)
323281 -> SockAddr
324− -> ServeConnection
282+ -> Application
325283 -> Counter
326284 -> InternalInfo
327285 -> IO ()
328−fork set mkConn addr serveConn counter ii0 = settingsFork set $ \ unmask ->
329− -- Run the connection maker to get a new connection, and ensure
330− -- that the connection is closed. If the mkConn call throws an
331− -- exception, we will leak the connection. If the mkConn call is
332− -- vulnerable to attacks (e.g., Slowloris), we do nothing to
333− -- protect the server. It is therefore vital that mkConn is well
334− -- vetted.
335− --
336− -- We grab the connection before registering timeouts since the
337− -- timeouts will be useless during connection creation, due to the
338− -- fact that async exceptions are still masked.
339− 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
340303
341304 -- We need to register a timeout handler for this thread, and
342305 -- cancel that handler as soon as we exit.
343− bracket (T.registerKillThread (timeoutManager ii0)) T.cancel $ \th ->
344−
345− let ii = ii0 { threadHandle = th }
346− -- We now have fully registered a connection close handler
347− -- in the case of all exceptions, so it is safe to one
348− -- again allow async exceptions.
349− in unmask .
350− -- Call the user-supplied on exception code if any
351− -- exceptions are thrown.
352− handle (settingsOnException set Nothing) .
353−
354− -- Call the user-supplied code for connection open and close events
355− bracket (onOpen addr) (onClose addr) $ \goingon ->
356−
357− -- Actually serve this connection.
358− -- bracket with closeConn above ensures the connection is closed.
359− when goingon $ serveConn conn ii addr transport set
360− where
361− 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
362320
363321 onOpen adr = increase counter >> settingsOnOpen set adr
364322 onClose adr _ = decrease counter >> settingsOnClose set adr
365323
366−-- The type of a function to serve a fully-prepared connection.
367−type ServeConnection = Connection
368− -> InternalInfo
369− -> SockAddr
370− -> Transport
371− -> Settings
372− -> IO ()
373−
374−-- Serve an HTTP\/2-unaware Application to a connection over any HTTP version.
375−serveDefault :: Application -> ServeConnection
376−serveDefault app = serveHTTP2 (promoteApplication app) app
377−
378−-- Serve an HTTP\/2-aware application over HTTP\/2 or a backup 'Application'
379−-- over HTTP\/1.1 or HTTP\/1.
380−serveHTTP2 :: HTTP2Application -> Application -> ServeConnection
381−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
382333 -- fixme: Upgrading to HTTP/2 should be supported.
383334 (h2,bs) <- if isHTTP2 transport then
384335 return (True, "")
@@ -388,19 +339,39 @@ serveHTTP2 app2 app conn ii origAddr transport settings = do
388339 return (True, bs0)
389340 else
390341 return (False, bs0)
342+ istatus <- newIORef False
391343 if settingsHTTP2Enabled settings && h2 then do
392− 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
393354 -- fixme: origAddr
394− http2 conn ii origAddr transport settings recvN app2
355+ checkTLS
356+ setConnHTTP2 conn True
357+ http2 conn transport ii origAddr settings recvN sendBS app
395358 else do
396− istatus <- newIORef False
397359 src <- mkSource (wrappedRecv conn th istatus (settingsSlowlorisSize settings))
398360 writeIORef istatus True
399361 leftoverSource src bs
400362 addr <- getProxyProtocolAddr src
401− http1 addr istatus src `E.catch` \e -> do
402− sendErrorResponse addr istatus e
403− 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+
404375 where
405376 getProxyProtocolAddr src =
406377 case settingsProxyProtocol settings of
@@ -442,30 +413,38 @@ serveHTTP2 app2 app conn ii origAddr transport settings = do
442413
443414 decodeAscii = map (chr . fromEnum) . S.unpack
444415
445− th = threadHandle ii
416+ shouldSendErrorResponse se
417+ | Just ConnectionClosedByPeer <- fromException se = False
418+ | otherwise = True
446419
447− sendErrorResponse addr istatus e = do
420+ sendErrorResponse req istatus e = do
448421 status <- readIORef istatus
449− when status $ void $
450− sendResponse
451− (settingsServerName settings)
452− 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
453426
454427 dummyreq addr = defaultRequest { remoteHost = addr }
455428
456429 errorResponse e = settingsOnExceptionResponse settings e
457430
458− http1 addr istatus src = do
459− (req', mremainingRef, idxhdr, nextBodyFlush) <- recvRequest settings conn ii addr src
460− 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
461433 keepAlive <- processRequest istatus src req mremainingRef idxhdr nextBodyFlush
462434 `E.catch` \e -> do
463− -- Call the user-supplied exception handlers, passing the request.
464− sendErrorResponse addr istatus e
465435 settingsOnException settings (Just req) e
466436 -- Don't throw the error again to prevent calling settingsOnException twice.
467437 return False
468− 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
469448
470449 processRequest istatus src req mremainingRef idxhdr nextBodyFlush = do
471450 -- Let the application run for as long as it wants
@@ -475,32 +454,38 @@ serveHTTP2 app2 app conn ii origAddr transport settings = do
475454 -- creating the request, we need to make sure that we don't get
476455 -- an async exception before calling the ResponseSource.
477456 keepAliveRef <- newIORef $ error "keepAliveRef not filled"
478− _ <- app req $ \res -> do
457+ r <- E.try $ app req $ \res -> do
479458 T.resume th
480459 -- FIXME consider forcing evaluation of the res here to
481460 -- send more meaningful error messages to the user.
482461 -- However, it may affect performance.
483462 writeIORef istatus False
484− keepAlive <- sendResponse
485− (settingsServerName settings)
486− conn ii req idxhdr (readSource src) res
463+ keepAlive <- sendResponse settings conn ii th req idxhdr (readSource src) res
487464 writeIORef keepAliveRef keepAlive
488465 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+
489475 keepAlive <- readIORef keepAliveRef
490476
491477 -- We just send a Response and it takes a time to
492478 -- receive a Request again. If we immediately call recv,
493− -- it is likely to fail and the IO manager works.
494− -- 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
495481 -- thread hoping that the next Request will arrive
496482 -- when this Haskell thread will be re-scheduled.
497483 -- This improves performance at least when
498484 -- the number of cores is small.
499485 Conc.yield
500486
501− if not keepAlive then
502− return False
503− else
487+ if keepAlive
488+ then
504489 -- If there is an unknown or large amount of data to still be read
505490 -- from the request body, simple drop this connection instead of
506491 -- reading it all in to satisfy a keep-alive request.
@@ -526,6 +511,21 @@ serveHTTP2 app2 app conn ii origAddr transport settings = do
526511 else
527512 return False
528513 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
529529
530530flushEntireBody :: IO ByteString -> IO ()
531531flushEntireBody src =
@@ -558,14 +558,42 @@ wrappedRecv Connection { connRecv = recv } th istatus slowlorisSize = do
558558 when (S.length bs >= slowlorisSize) $ T.tickle th
559559 return bs
560560
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+--
561576-- Copied from: https://github.com/mzero/plush/blob/master/src/Plush/Server/Warp.hs
577+--
578+-- @since 3.2.17
562579setSocketCloseOnExec :: Socket -> IO ()
563580#if WINDOWS
564581setSocketCloseOnExec _ = return ()
565582#else
566−setSocketCloseOnExec socket =
567− 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
568590#endif
569591
570−gracefulShutdown :: Counter -> IO ()
571−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