@@ -1,29 +1,29 @@
1−{-# LANGUAGE AllowAmbiguousTypes #-}
1+{-# LANGUAGE CPP #-}
22{-# LANGUAGE ConstraintKinds #-}
33{-# LANGUAGE DataKinds #-}
44{-# LANGUAGE DeriveDataTypeable #-}
55{-# LANGUAGE FlexibleContexts #-}
66{-# LANGUAGE FlexibleInstances #-}
7−{-# LANGUAGE InstanceSigs #-}
87{-# LANGUAGE MultiParamTypeClasses #-}
98{-# LANGUAGE OverloadedStrings #-}
109{-# LANGUAGE PolyKinds #-}
11−{-# LANGUAGE QuantifiedConstraints #-}
1210{-# LANGUAGE RankNTypes #-}
1311{-# LANGUAGE ScopedTypeVariables #-}
1412{-# LANGUAGE TupleSections #-}
15−{-# LANGUAGE TypeApplications #-}
1613{-# LANGUAGE TypeFamilies #-}
1714{-# LANGUAGE TypeOperators #-}
1815{-# LANGUAGE UndecidableInstances #-}
1916
17+#if MIN_VERSION_base(4,9,0) && __GLASGOW_HASKELL__ >= 802
18+#define HAS_TYPE_ERROR
19+#endif
20+
2021module Servant.Server.Internal
2122 ( module Servant.Server.Internal
2223 , module Servant.Server.Internal.BasicAuth
2324 , module Servant.Server.Internal.Context
2425 , module Servant.Server.Internal.Delayed
2526 , module Servant.Server.Internal.DelayedIO
26− , module Servant.Server.Internal.ErrorFormatter
2727 , module Servant.Server.Internal.Handler
2828 , module Servant.Server.Internal.Router
2929 , module Servant.Server.Internal.RouteResult
@@ -41,11 +41,12 @@ import qualified Data.ByteString as B
4141import qualified Data.ByteString.Builder as BB
4242import qualified Data.ByteString.Char8 as BC8
4343import qualified Data.ByteString.Lazy as BL
44−import Data.Constraint (Constraint, Dict(..))
4544import Data.Either
4645 (partitionEithers)
4746import Data.Maybe
4847 (fromMaybe, isNothing, mapMaybe, maybeToList)
48+import Data.Semigroup
49+ ((<>))
4950import Data.String
5051 (IsString (..))
5152import Data.String.Conversions
@@ -54,9 +55,8 @@ import Data.Tagged
5455 (Tagged (..), retag, untag)
5556import qualified Data.Text as T
5657import Data.Typeable
57−import GHC.Generics
5858import GHC.TypeLits
59− (KnownNat, KnownSymbol, TypeError, symbolVal)
59+ (KnownNat, KnownSymbol, natVal, symbolVal)
6060import qualified Network.HTTP.Media as NHM
6161import Network.HTTP.Types hiding
6262 (Header, ResponseHeaders)
@@ -64,52 +64,45 @@ import Network.Socket
6464 (SockAddr)
6565import Network.Wai
6666 (Application, Request, httpVersion, isSecure, lazyRequestBody,
67− queryString, remoteHost, getRequestBodyChunk, requestHeaders,
67+ rawQueryString, remoteHost, requestBody, requestHeaders,
6868 requestMethod, responseLBS, responseStream, vault)
6969import Prelude ()
7070import Prelude.Compat
7171import Servant.API
7272 ((:<|>) (..), (:>), Accept (..), BasicAuth, Capture',
73− CaptureAll, Description, EmptyAPI, Fragment,
74− FramingRender (..), FramingUnrender (..), FromSourceIO (..),
75− Header', If, IsSecure (..), NoContentVerb, QueryFlag,
76− QueryParam', QueryParams, Raw, ReflectMethod (reflectMethod),
77− RemoteHost, ReqBody', SBool (..), SBoolI (..), SourceIO,
78− Stream, StreamBody', Summary, ToSourceIO (..), Vault, Verb,
79− WithNamedContext, NamedRoutes)
80−import Servant.API.Generic (GenericMode(..), ToServant, ToServantApi, GServantProduct, toServant, fromServant)
73+ CaptureAll, Description, EmptyAPI, FramingRender (..),
74+ FramingUnrender (..), FromSourceIO (..), Header', If,
75+ IsSecure (..), QueryFlag, QueryParam', QueryParams, Raw,
76+ ReflectMethod (reflectMethod), RemoteHost, ReqBody',
77+ SBool (..), SBoolI (..), SourceIO, Stream, StreamBody',
78+ Summary, ToSourceIO (..), Vault, Verb, WithNamedContext)
8179import Servant.API.ContentTypes
8280 (AcceptHeader (..), AllCTRender (..), AllCTUnrender (..),
83− AllMime, MimeRender (..), MimeUnrender (..), NoContent,
84− canHandleAcceptH)
81+ AllMime, MimeRender (..), MimeUnrender (..), canHandleAcceptH)
8582import Servant.API.Modifiers
8683 (FoldLenient, FoldRequired, RequestArgument,
8784 unfoldRequestArgument)
8885import Servant.API.ResponseHeaders
8986 (GetHeaders, Headers, getHeaders, getResponse)
90−import Servant.API.Status
91− (statusFromNat)
9287import qualified Servant.Types.SourceT as S
93−import Servant.API.TypeErrors
9488import Web.HttpApiData
95− (FromHttpApiData, parseHeader, parseQueryParam, parseUrlPiece,
96− parseUrlPieces)
89+ (FromHttpApiData, parseHeader, parseQueryParam,
90+ parseUrlPieceMaybe, parseUrlPieces, parseUrlPiece)
9791
9892import Servant.Server.Internal.BasicAuth
9993import Servant.Server.Internal.Context
10094import Servant.Server.Internal.Delayed
10195import Servant.Server.Internal.DelayedIO
102−import Servant.Server.Internal.ErrorFormatter
10396import Servant.Server.Internal.Handler
10497import Servant.Server.Internal.Router
10598import Servant.Server.Internal.RouteResult
10699import Servant.Server.Internal.RoutingApplication
107100import Servant.Server.Internal.ServerError
108101
102+#ifdef HAS_TYPE_ERROR
109103import GHC.TypeLits
110104 (ErrorMessage (..), TypeError)
111−import Servant.API.TypeLevel
112− (AtLeastOneFragment, FragmentUnique)
105+#endif
113106
114107class HasServer api context where
115108 type ServerT api (m :: * -> *) :: *
@@ -173,10 +166,7 @@ instance (HasServer a context, HasServer b context) => HasServer (a :<|> b) cont
173166-- > server = getBook
174167-- > where getBook :: Text -> Handler Book
175168-- > getBook isbn = ...
176−instance (KnownSymbol capture, FromHttpApiData a, Typeable a
177− , HasServer api context, SBoolI (FoldLenient mods)
178− , HasContextEntry (MkContextWithErrorFormatter context) ErrorFormatters
179− )
169+instance (KnownSymbol capture, FromHttpApiData a, HasServer api context, SBoolI (FoldLenient mods))
180170 => HasServer (Capture' mods capture a :> api) context where
181171
182172 type ServerT (Capture' mods capture a :> api) m =
@@ -185,19 +175,15 @@ instance (KnownSymbol capture, FromHttpApiData a, Typeable a
185175 hoistServerWithContext _ pc nt s = hoistServerWithContext (Proxy :: Proxy api) pc nt . s
186176
187177 route Proxy context d =
188− CaptureRouter [hint] $
178+ CaptureRouter $
189179 route (Proxy :: Proxy api)
190180 context
191− (addCapture d $ \ txt -> withRequest $ \ request ->
192− case ( sbool :: SBool (FoldLenient mods)
193− , parseUrlPiece txt :: Either T.Text a) of
194− (SFalse, Left e) -> delayedFail $ formatError rep request $ cs e
195− (SFalse, Right v) -> return v
196− (STrue, piece) -> return $ (either (Left . cs) Right) piece)
197− where
198− rep = typeRep (Proxy :: Proxy Capture')
199− formatError = urlParseErrorFormatter $ getContextEntry (mkContextWithErrorFormatter context)
200− hint = CaptureHint (T.pack $ symbolVal $ Proxy @capture) (typeRep (Proxy :: Proxy a))
181+ (addCapture d $ \ txt -> case ( sbool :: SBool (FoldLenient mods)
182+ , parseUrlPiece txt :: Either T.Text a) of
183+ (SFalse, Left e) -> delayedFail err400 { errBody = cs e }
184+ (SFalse, Right v) -> return v
185+ (STrue, piece) -> return $ (either (Left . cs) Right) piece
186+ )
201187
202188-- | If you use 'CaptureAll' in one of the endpoints for your API,
203189-- this automatically requires your server-side handler to be a
@@ -216,10 +202,7 @@ instance (KnownSymbol capture, FromHttpApiData a, Typeable a
216202-- > server = getSourceFile
217203-- > where getSourceFile :: [Text] -> Handler Book
218204-- > getSourceFile pathSegments = ...
219−instance (KnownSymbol capture, FromHttpApiData a, Typeable a
220− , HasServer api context
221− , HasContextEntry (MkContextWithErrorFormatter context) ErrorFormatters
222− )
205+instance (KnownSymbol capture, FromHttpApiData a, HasServer api context)
223206 => HasServer (CaptureAll capture a :> api) context where
224207
225208 type ServerT (CaptureAll capture a :> api) m =
@@ -228,18 +211,14 @@ instance (KnownSymbol capture, FromHttpApiData a, Typeable a
228211 hoistServerWithContext _ pc nt s = hoistServerWithContext (Proxy :: Proxy api) pc nt . s
229212
230213 route Proxy context d =
231− CaptureAllRouter [hint] $
214+ CaptureAllRouter $
232215 route (Proxy :: Proxy api)
233216 context
234− (addCapture d $ \ txts -> withRequest $ \ request ->
235− case parseUrlPieces txts of
236− Left e -> delayedFail $ formatError rep request $ cs e
237− Right v -> return v
217+ (addCapture d $ \ txts -> case parseUrlPieces txts of
218+ Left _ -> delayedFail err400
219+ Right v -> return v
238220 )
239− where
240− rep = typeRep (Proxy :: Proxy CaptureAll)
241− formatError = urlParseErrorFormatter $ getContextEntry (mkContextWithErrorFormatter context)
242− hint = CaptureHint (T.pack $ symbolVal $ Proxy @capture) (typeRep (Proxy :: Proxy [a]))
221+
243222
244223allowedMethodHead :: Method -> Request -> Bool
245224allowedMethodHead method request = method == methodGet && requestMethod request == methodHead
@@ -259,10 +238,10 @@ methodCheck method request
259238-- body check is no longer an option. However, we now run the accept
260239-- check before the body check and can therefore afford to make it
261240-- recoverable.
262−acceptCheck :: (AllMime list) => Proxy list -> AcceptHeader -> DelayedIO ()
241+acceptCheck :: (AllMime list) => Proxy list -> B.ByteString -> DelayedIO ()
263242acceptCheck proxy accH
264− | canHandleAcceptH proxy accH = return ()
265− | otherwise = delayedFail err406
243+ | canHandleAcceptH proxy (AcceptHeader accH) = return ()
244+ | otherwise = delayedFail err406
266245
267246methodRouter :: (AllCTRender ctypes a)
268247 => (b -> ([(HeaderName, B.ByteString)], a))
@@ -272,28 +251,17 @@ methodRouter :: (AllCTRender ctypes a)
272251methodRouter splitHeaders method proxy status action = leafRouter route'
273252 where
274253 route' env request respond =
275− let accH = getAcceptHeader request
254+ let accH = fromMaybe ct_wildcard $ lookup hAccept $ requestHeaders request
276255 in runAction (action `addMethodCheck` methodCheck method request
277256 `addAcceptCheck` acceptCheck proxy accH
278257 ) env request respond $ \ output -> do
279258 let (headers, b) = splitHeaders output
280− case handleAcceptH proxy accH b of
259+ case handleAcceptH proxy (AcceptHeader accH) b of
281260 Nothing -> FailFatal err406 -- this should not happen (checked before), so we make it fatal if it does
282261 Just (contentT, body) ->
283262 let bdy = if allowedMethodHead method request then "" else body
284263 in Route $ responseLBS status ((hContentType, cs contentT) : headers) bdy
285264
286−noContentRouter :: Method
287− -> Status
288− -> Delayed env (Handler b)
289− -> Router env
290−noContentRouter method status action = leafRouter route'
291− where
292− route' env request respond =
293− runAction (action `addMethodCheck` methodCheck method request)
294− env request respond $ \ _output ->
295− Route $ responseLBS status [] ""
296−
297265instance {-# OVERLAPPABLE #-}
298266 ( AllCTRender ctypes a, ReflectMethod method, KnownNat status
299267 ) => HasServer (Verb method status ctypes a) context where
@@ -303,7 +271,7 @@ instance {-# OVERLAPPABLE #-}
303271
304272 route Proxy _ = methodRouter ([],) method (Proxy :: Proxy ctypes) status
305273 where method = reflectMethod (Proxy :: Proxy method)
306− status = statusFromNat (Proxy :: Proxy status)
274+ status = toEnum . fromInteger $ natVal (Proxy :: Proxy status)
307275
308276instance {-# OVERLAPPING #-}
309277 ( AllCTRender ctypes a, ReflectMethod method, KnownNat status
@@ -315,16 +283,8 @@ instance {-# OVERLAPPING #-}
315283
316284 route Proxy _ = methodRouter (\x -> (getHeaders x, getResponse x)) method (Proxy :: Proxy ctypes) status
317285 where method = reflectMethod (Proxy :: Proxy method)
318− status = statusFromNat (Proxy :: Proxy status)
319−
320−instance (ReflectMethod method) =>
321− HasServer (NoContentVerb method) context where
286+ status = toEnum . fromInteger $ natVal (Proxy :: Proxy status)
322287
323− type ServerT (NoContentVerb method) m = m NoContent
324− hoistServerWithContext _ _ nt s = nt s
325−
326− route Proxy _ = noContentRouter method status204
327− where method = reflectMethod (Proxy :: Proxy method)
328288
329289instance {-# OVERLAPPABLE #-}
330290 ( MimeRender ctype chunk, ReflectMethod method, KnownNat status,
@@ -336,7 +296,7 @@ instance {-# OVERLAPPABLE #-}
336296
337297 route Proxy _ = streamRouter ([],) method status (Proxy :: Proxy framing) (Proxy :: Proxy ctype)
338298 where method = reflectMethod (Proxy :: Proxy method)
339− status = statusFromNat (Proxy :: Proxy status)
299+ status = toEnum . fromInteger $ natVal (Proxy :: Proxy status)
340300
341301
342302instance {-# OVERLAPPING #-}
@@ -350,7 +310,7 @@ instance {-# OVERLAPPING #-}
350310
351311 route Proxy _ = streamRouter (\x -> (getHeaders x, getResponse x)) method status (Proxy :: Proxy framing) (Proxy :: Proxy ctype)
352312 where method = reflectMethod (Proxy :: Proxy method)
353− status = statusFromNat (Proxy :: Proxy status)
313+ status = toEnum . fromInteger $ natVal (Proxy :: Proxy status)
354314
355315
356316streamRouter :: forall ctype a c chunk env framing. (MimeRender ctype chunk, FramingRender framing, ToSourceIO chunk a) =>
@@ -362,7 +322,7 @@ streamRouter :: forall ctype a c chunk env framing. (MimeRender ctype chunk, Fra
362322 -> Delayed env (Handler c)
363323 -> Router env
364324streamRouter splitHeaders method status framingproxy ctypeproxy action = leafRouter $ \env request respond ->
365− let AcceptHeader accH = getAcceptHeader request
325+ let accH = fromMaybe ct_wildcard $ lookup hAccept $ requestHeaders request
366326 cmediatype = NHM.matchAccept [contentType ctypeproxy] accH
367327 accCheck = when (isNothing cmediatype) $ delayedFail err406
368328 contentHeader = (hContentType, NHM.renderHeader . maybeToList $ cmediatype)
@@ -407,7 +367,6 @@ streamRouter splitHeaders method status framingproxy ctypeproxy action = leafRou
407367instance
408368 (KnownSymbol sym, FromHttpApiData a, HasServer api context
409369 , SBoolI (FoldRequired mods), SBoolI (FoldLenient mods)
410− , HasContextEntry (MkContextWithErrorFormatter context) ErrorFormatters
411370 )
412371 => HasServer (Header' mods sym a :> api) context where
413372------
@@ -419,9 +378,6 @@ instance
419378 route Proxy context subserver = route (Proxy :: Proxy api) context $
420379 subserver `addHeaderCheck` withRequest headerCheck
421380 where
422− rep = typeRep (Proxy :: Proxy Header')
423− formatError = headerParseErrorFormatter $ getContextEntry (mkContextWithErrorFormatter context)
424−
425381 headerName :: IsString n => n
426382 headerName = fromString $ symbolVal (Proxy :: Proxy sym)
427383
@@ -432,13 +388,15 @@ instance
432388 mev :: Maybe (Either T.Text a)
433389 mev = fmap parseHeader $ lookup headerName (requestHeaders req)
434390
435− errReq = delayedFailFatal $ formatError rep req
436− $ "Header " <> headerName <> " is required"
391+ errReq = delayedFailFatal err400
392+ { errBody = "Header " <> headerName <> " is required"
393+ }
437394
438− errSt e = delayedFailFatal $ formatError rep req
439− $ cs $ "Error parsing header "
440− <> headerName
441− <> " failed: " <> e
395+ errSt e = delayedFailFatal err400
396+ { errBody = cs $ "Error parsing header "
397+ <> headerName
398+ <> " failed: " <> e
399+ }
442400
443401-- | If you use @'QueryParam' "author" Text@ in one of the endpoints for your API,
444402-- this automatically requires your server-side handler to be a function
@@ -464,7 +422,6 @@ instance
464422instance
465423 ( KnownSymbol sym, FromHttpApiData a, HasServer api context
466424 , SBoolI (FoldRequired mods), SBoolI (FoldLenient mods)
467− , HasContextEntry (MkContextWithErrorFormatter context) ErrorFormatters
468425 )
469426 => HasServer (QueryParam' mods sym a :> api) context where
470427------
@@ -474,12 +431,9 @@ instance
474431 hoistServerWithContext _ pc nt s = hoistServerWithContext (Proxy :: Proxy api) pc nt . s
475432
476433 route Proxy context subserver =
477− let querytext = queryToQueryText . queryString
434+ let querytext req = parseQueryText $ rawQueryString req
478435 paramname = cs $ symbolVal (Proxy :: Proxy sym)
479436
480− rep = typeRep (Proxy :: Proxy QueryParam')
481− formatError = urlParseErrorFormatter $ getContextEntry (mkContextWithErrorFormatter context)
482−
483437 parseParam :: Request -> DelayedIO (RequestArgument mods a)
484438 parseParam req =
485439 unfoldRequestArgument (Proxy :: Proxy mods) errReq errSt mev
@@ -487,12 +441,14 @@ instance
487441 mev :: Maybe (Either T.Text a)
488442 mev = fmap parseQueryParam $ join $ lookup paramname $ querytext req
489443
490− errReq = delayedFailFatal $ formatError rep req
491− $ cs $ "Query parameter " <> paramname <> " is required"
444+ errReq = delayedFailFatal err400
445+ { errBody = cs $ "Query parameter " <> paramname <> " is required"
446+ }
492447
493− errSt e = delayedFailFatal $ formatError rep req
494− $ cs $ "Error parsing query parameter "
495− <> paramname <> " failed: " <> e
448+ errSt e = delayedFailFatal err400
449+ { errBody = cs $ "Error parsing query parameter "
450+ <> paramname <> " failed: " <> e
451+ }
496452
497453 delayed = addParameterCheck subserver . withRequest $ \req ->
498454 parseParam req
@@ -518,8 +474,7 @@ instance
518474-- > server = getBooksBy
519475-- > where getBooksBy :: [Text] -> Handler [Book]
520476-- > getBooksBy authors = ...return all books by these authors...
521−instance (KnownSymbol sym, FromHttpApiData a, HasServer api context
522− , HasContextEntry (MkContextWithErrorFormatter context) ErrorFormatters)
477+instance (KnownSymbol sym, FromHttpApiData a, HasServer api context)
523478 => HasServer (QueryParams sym a :> api) context where
524479
525480 type ServerT (QueryParams sym a :> api) m =
@@ -530,23 +485,21 @@ instance (KnownSymbol sym, FromHttpApiData a, HasServer api context
530485 route Proxy context subserver = route (Proxy :: Proxy api) context $
531486 subserver `addParameterCheck` withRequest paramsCheck
532487 where
533− rep = typeRep (Proxy :: Proxy QueryParams)
534− formatError = urlParseErrorFormatter $ getContextEntry (mkContextWithErrorFormatter context)
535−
536488 paramname = cs $ symbolVal (Proxy :: Proxy sym)
537489 paramsCheck req =
538490 case partitionEithers $ fmap parseQueryParam params of
539491 ([], parsed) -> return parsed
540− (errs, _) -> delayedFailFatal $ formatError rep req
541− $ cs $ "Error parsing query parameter(s) "
542− <> paramname <> " failed: "
543− <> T.intercalate ", " errs
492+ (errs, _) -> delayedFailFatal err400
493+ { errBody = cs $ "Error parsing query parameter(s) "
494+ <> paramname <> " failed: "
495+ <> T.intercalate ", " errs
496+ }
544497 where
545498 params :: [T.Text]
546499 params = mapMaybe snd
547500 . filter (looksLikeParam . fst)
548− . queryToQueryText
549− . queryString
501+ . parseQueryText
502+ . rawQueryString
550503 $ req
551504
552505 looksLikeParam name = name == paramname || name == (paramname <> "[]")
@@ -572,7 +525,7 @@ instance (KnownSymbol sym, HasServer api context)
572525 hoistServerWithContext _ pc nt s = hoistServerWithContext (Proxy :: Proxy api) pc nt . s
573526
574527 route Proxy context subserver =
575− let querytext = queryToQueryText . queryString
528+ let querytext r = parseQueryText $ rawQueryString r
576529 param r = case lookup paramname (querytext r) of
577530 Just Nothing -> True -- param is there, with no value
578531 Just (Just v) -> examine v -- param with a value
@@ -614,7 +567,7 @@ instance HasServer Raw context where
614567-- The @Content-Type@ header is inspected, and the list provided is used to
615568-- attempt deserialization. If the request does not have a @Content-Type@
616569-- header, it is treated as @application/octet-stream@ (as specified in
617−-- [RFC 7231 section 3.1.1.5](http://tools.ietf.org/html/rfc7231#section-3.1.1.5)).
570+-- <http://tools.ietf.org/html/rfc7231#section-3.1.1.5 RFC7231>.
618571-- This lets servant worry about extracting it from the request and turning
619572-- it into a value of the type you specify.
620573--
@@ -630,7 +583,6 @@ instance HasServer Raw context where
630583-- > where postBook :: Book -> Handler Book
631584-- > postBook book = ...insert into your db...
632585instance ( AllCTUnrender list a, HasServer api context, SBoolI (FoldLenient mods)
633− , HasContextEntry (MkContextWithErrorFormatter context) ErrorFormatters
634586 ) => HasServer (ReqBody' mods list a :> api) context where
635587
636588 type ServerT (ReqBody' mods list a :> api) m =
@@ -642,9 +594,6 @@ instance ( AllCTUnrender list a, HasServer api context, SBoolI (FoldLenient mods
642594 = route (Proxy :: Proxy api) context $
643595 addBodyCheck subserver ctCheck bodyCheck
644596 where
645− rep = typeRep (Proxy :: Proxy ReqBody')
646− formatError = bodyParserErrorFormatter $ getContextEntry (mkContextWithErrorFormatter context)
647−
648597 -- Content-Type check, we only lookup we can try to parse the request body
649598 ctCheck = withRequest $ \ request -> do
650599 -- See HTTP RFC 2616, section 7.2.1
@@ -663,7 +612,7 @@ instance ( AllCTUnrender list a, HasServer api context, SBoolI (FoldLenient mods
663612 case sbool :: SBool (FoldLenient mods) of
664613 STrue -> return mrqbody
665614 SFalse -> case mrqbody of
666− Left e -> delayedFailFatal $ formatError rep request e
615+ Left e -> delayedFailFatal err400 { errBody = cs e }
667616 Right v -> return v
668617
669618instance
@@ -686,7 +635,7 @@ instance
686635 bodyCheck fromRS = withRequest $ \req -> do
687636 let mimeUnrender' = mimeUnrender (Proxy :: Proxy ctype) :: BL.ByteString -> Either String chunk
688637 let framingUnrender' = framingUnrender (Proxy :: Proxy framing) mimeUnrender' :: SourceIO B.ByteString -> SourceIO chunk
689− let body = getRequestBodyChunk req
638+ let body = requestBody req
690639 let rs = S.fromAction B.null body
691640 let rs' = fromRS $ framingUnrender' rs
692641 return rs'
@@ -755,12 +704,12 @@ data EmptyServer = EmptyServer deriving (Typeable, Eq, Show, Bounded, Enum)
755704emptyServer :: ServerT EmptyAPI m
756705emptyServer = Tagged EmptyServer
757706
758−-- | The server for an `EmptyAPI` is `emptyServer`.
707+-- | The server for an `EmptyAPI` is `emptyAPIServer`.
759708--
760709-- > type MyApi = "nothing" :> EmptyApi
761710-- >
762711-- > server :: Server MyApi
763−-- > server = emptyServer
712+-- > server = emptyAPIServer
764713instance HasServer EmptyAPI context where
765714 type ServerT EmptyAPI m = Tagged m EmptyServer
766715
@@ -789,10 +738,7 @@ instance ( KnownSymbol realm
789738-- * helpers
790739
791740ct_wildcard :: B.ByteString
792−ct_wildcard = "*" <> "/" <> "*"
793−
794−getAcceptHeader :: Request -> AcceptHeader
795−getAcceptHeader = AcceptHeader . fromMaybe ct_wildcard . lookup hAccept . requestHeaders
741+ct_wildcard = "*" <> "/" <> "*" -- Because CPP
796742
797743-- * General Authentication
798744
@@ -817,15 +763,39 @@ instance (HasContextEntry context (NamedContext name subContext), HasServer subA
817763 hoistServerWithContext _ _ nt s = hoistServerWithContext (Proxy :: Proxy subApi) (Proxy :: Proxy subContext) nt s
818764
819765-------------------------------------------------------------------------------
820−-- Custom type errors
766+-- TypeError helpers
821767-------------------------------------------------------------------------------
822768
823−-- Erroring instance for 'HasServer' when a combinator is not fully applied
824−instance TypeError (PartialApplication HasServer arr) => HasServer ((arr :: a -> b) :> sub) context
769+#ifdef HAS_TYPE_ERROR
770+-- | This instance catches mistakes when there are non-saturated
771+-- type applications on LHS of ':>'.
772+--
773+-- >>> serve (Proxy :: Proxy (Capture "foo" :> Get '[JSON] Int)) (error "...")
774+-- ...
775+-- ...Expected something of kind Symbol or *, got: k -> l on the LHS of ':>'.
776+-- ...Maybe you haven't applied enough arguments to
777+-- ...Capture' '[] "foo"
778+-- ...
779+--
780+-- >>> undefined :: Server (Capture "foo" :> Get '[JSON] Int)
781+-- ...
782+-- ...Expected something of kind Symbol or *, got: k -> l on the LHS of ':>'.
783+-- ...Maybe you haven't applied enough arguments to
784+-- ...Capture' '[] "foo"
785+-- ...
786+--
787+instance TypeError (HasServerArrowKindError arr) => HasServer ((arr :: k -> l) :> api) context
825788 where
826− type ServerT (arr :> sub) _ = TypeError (PartialApplication (HasServer :: * -> [*] -> Constraint) arr)
827− route = error "unreachable"
828− hoistServerWithContext _ _ _ _ = error "unreachable"
789+ type ServerT (arr :> api) m = TypeError (HasServerArrowKindError arr)
790+ -- it doens't really matter what sub route we peak
791+ route _ _ _ = error "servant-server panic: impossible happened in HasServer (arr :> api)"
792+ hoistServerWithContext _ _ _ = id
793+
794+-- Cannot have TypeError here, otherwise use of this symbol will error :)
795+type HasServerArrowKindError arr =
796+ 'Text "Expected something of kind Symbol or *, got: k -> l on the LHS of ':>'."
797+ ':$$: 'Text "Maybe you haven't applied enough arguments to"
798+ ':$$: 'ShowType arr
829799
830800-- | This instance prevents from accidentally using '->' instead of ':>'
831801--
@@ -859,103 +829,7 @@ type HasServerArrowTypeError a b =
859829 ':$$: 'ShowType a
860830 ':$$: 'Text "and"
861831 ':$$: 'ShowType b
862−
863−-- Erroring instances for 'HasServer' for unknown API combinators
864−
865−-- XXX: This omits the @context@ parameter, e.g.:
866−--
867−-- "There is no instance for HasServer (Bool :> …)". Do we care ?
868−instance {-# OVERLAPPABLE #-} TypeError (NoInstanceForSub HasServer ty) => HasServer (ty :> sub) context
869−
870−instance {-# OVERLAPPABLE #-} TypeError (NoInstanceFor (HasServer api context)) => HasServer api context
871−
872−-- | Ignore @'Fragment'@ in server handlers.
873−-- See <https://ietf.org/rfc/rfc2616.html#section-15.1.3> for more details.
874−--
875−-- Example:
876−--
877−-- > type MyApi = "books" :> Fragment Text :> Get '[JSON] [Book]
878−-- >
879−-- > server :: Server MyApi
880−-- > server = getBooks
881−-- > where getBooks :: Handler [Book]
882−-- > getBooks = ...return all books...
883−instance (AtLeastOneFragment api, FragmentUnique (Fragment a1 :> api), HasServer api context)
884− => HasServer (Fragment a1 :> api) context where
885− type ServerT (Fragment a1 :> api) m = ServerT api m
886−
887− route _ = route (Proxy :: Proxy api)
888−
889− hoistServerWithContext _ = hoistServerWithContext (Proxy :: Proxy api)
832+#endif
890833
891834-- $setup
892835-- >>> import Servant
893−
894−-- | A type that specifies that an API record contains a server implementation.
895−data AsServerT (m :: * -> *)
896−instance GenericMode (AsServerT m) where
897− type AsServerT m :- api = ServerT api m
898−
899−type AsServer = AsServerT Handler
900−
901−
902−-- | Set of constraints required to convert to / from vanilla server types.
903−type GServerConstraints api m =
904− ( ToServant api (AsServerT m) ~ ServerT (ToServantApi api) m
905− , GServantProduct (Rep (api (AsServerT m)))
906− )
907−
908−-- | This class is a necessary evil: in the implementation of 'HasServer' for
909−-- @'NamedRoutes' api@, we essentially need the quantified constraint @forall
910−-- m. 'GServerConstraints' m@ to hold.
911−--
912−-- We cannot require do that directly as the definition of 'GServerConstraints'
913−-- contains type family applications ('Rep' and 'ServerT'). The trick is to hide
914−-- those type family applications behind a typeclass providing evidence for
915−-- @'GServerConstraints' api m@ in the form of a dictionary, and require that
916−-- @forall m. 'GServer' api m@ instead.
917−--
918−-- Users shouldn't have to worry about this class, as the only possible instance
919−-- is provided in this module for all record APIs.
920−
921−class GServer (api :: * -> *) (m :: * -> *) where
922− gServerProof :: Dict (GServerConstraints api m)
923−
924−instance
925− ( ToServant api (AsServerT m) ~ ServerT (ToServantApi api) m
926− , GServantProduct (Rep (api (AsServerT m)))
927− ) => GServer api m where
928− gServerProof = Dict
929−
930−instance
931− ( HasServer (ToServantApi api) context
932− , forall m. Generic (api (AsServerT m))
933− , forall m. GServer api m
934− ) => HasServer (NamedRoutes api) context where
935−
936− type ServerT (NamedRoutes api) m = api (AsServerT m)
937−
938− route
939− :: Proxy (NamedRoutes api)
940− -> Context context
941− -> Delayed env (api (AsServerT Handler))
942− -> Router env
943− route _ ctx delayed =
944− case gServerProof @api @Handler of
945− Dict -> route (Proxy @(ToServantApi api)) ctx (toServant <$> delayed)
946−
947− hoistServerWithContext
948− :: forall m n. Proxy (NamedRoutes api)
949− -> Proxy context
950− -> (forall x. m x -> n x)
951− -> api (AsServerT m)
952− -> api (AsServerT n)
953− hoistServerWithContext _ pctx nat server =
954− case (gServerProof @api @m, gServerProof @api @n) of
955− (Dict, Dict) ->
956− fromServant servantSrvN
957− where
958− servantSrvM :: ServerT (ToServantApi api) m =
959− toServant server
960− servantSrvN :: ServerT (ToServantApi api) n =
961− hoistServerWithContext (Proxy @(ToServantApi api)) pctx nat servantSrvM