This commit is contained in:
Michael Snoyman 2012-02-09 09:07:53 +02:00
parent a26ad237dd
commit 4dd9880389
15 changed files with 50 additions and 23 deletions

View File

@ -8,6 +8,8 @@ module Yesod.Core
-- ** Breadcrumbs -- ** Breadcrumbs
, YesodBreadcrumbs (..) , YesodBreadcrumbs (..)
, breadcrumbs , breadcrumbs
-- * Types
, Approot (..)
-- * Utitlities -- * Utitlities
, maybeAuthorized , maybeAuthorized
, widgetToPageContent , widgetToPageContent

View File

@ -182,7 +182,7 @@ sendRedirect y segments' env =
, ("Location", Blaze.ByteString.Builder.toByteString dest') , ("Location", Blaze.ByteString.Builder.toByteString dest')
] "Redirecting" ] "Redirecting"
where where
dest = joinPath y (approot y) segments' [] dest = joinPath y (resolveApproot y env) segments' []
dest' = dest' =
if S.null (W.rawQueryString env) if S.null (W.rawQueryString env)
then dest then dest

View File

@ -28,6 +28,8 @@ module Yesod.Internal.Core
-- * Misc -- * Misc
, yesodVersion , yesodVersion
, yesodRender , yesodRender
, resolveApproot
, Approot (..)
) where ) where
import Yesod.Content import Yesod.Content
@ -121,20 +123,36 @@ class YesodDispatch sub master where
-> W.Application -> W.Application
yesodRunner = defaultYesodRunner yesodRunner = defaultYesodRunner
-- | Define settings for a Yesod applications. The only required setting is -- | How to determine the root of the application for constructing URLs.
-- 'approot'; other than that, there are intelligent defaults. --
-- Note that future versions of Yesod may add new constructors without bumping
-- the major version number. As a result, you should /not/ pattern match on
-- @Approot@ values.
data Approot master = ApprootRelative -- ^ No application root.
| ApprootStatic Text
| ApprootMaster (master -> Text)
| ApprootRequest (master -> W.Request -> Text)
type ResolvedApproot = Text
-- | Define settings for a Yesod applications. All methods have intelligent
-- defaults, and therefore no implementation is required.
class RenderRoute a => Yesod a where class RenderRoute a => Yesod a where
-- | An absolute URL to the root of the application. Do not include -- | An absolute URL to the root of the application. Do not include
-- trailing slash. -- trailing slash.
-- --
-- If you want to be lazy, you can supply an empty string under the -- Default value: 'ApprootRelative'. This is valid under the following
-- following conditions: -- conditions:
-- --
-- * Your application is served from the root of the domain. -- * Your application is served from the root of the domain.
-- --
-- * You do not use any features that require absolute URLs, such as Atom -- * You do not use any features that require absolute URLs, such as Atom
-- feeds and XML sitemaps. -- feeds and XML sitemaps.
approot :: a -> Text --
-- If this is not true, you should override with a different
-- implementation.
approot :: Approot a
approot = ApprootRelative
-- | The encryption key to be used for encrypting client sessions. -- | The encryption key to be used for encrypting client sessions.
-- Returning 'Nothing' disables sessions. -- Returning 'Nothing' disables sessions.
@ -395,7 +413,8 @@ defaultYesodRunner handler master sub murl toMasterRoute mkey req = do
handler handler
let sessionMap = Map.fromList let sessionMap = Map.fromList
$ filter (\(x, _) -> x /= nonceKey) session' $ filter (\(x, _) -> x /= nonceKey) session'
yar <- handlerToYAR master sub toMasterRoute (yesodRender master) errorHandler rr murl sessionMap h let ra = resolveApproot master req
yar <- handlerToYAR master sub toMasterRoute (yesodRender master ra) errorHandler rr murl sessionMap h
let mnonce = reqNonce rr let mnonce = reqNonce rr
-- FIXME should we be caching this IV value and reusing it for efficiency? -- FIXME should we be caching this IV value and reusing it for efficiency?
iv <- {-# SCC "iv" #-} maybe (return $ error "Should not be used") (const $ liftIO CS.randomIV) mkey iv <- {-# SCC "iv" #-} maybe (return $ error "Should not be used") (const $ liftIO CS.randomIV) mkey
@ -633,14 +652,23 @@ ynHelper render scripts jscript jsLoc =
yesodRender :: Yesod y yesodRender :: Yesod y
=> y => y
-> ResolvedApproot
-> Route y -> Route y
-> [(Text, Text)] -- ^ url query string -> [(Text, Text)] -- ^ url query string
-> Text -> Text
yesodRender y url params = yesodRender y ar url params =
TE.decodeUtf8 $ toByteString $ TE.decodeUtf8 $ toByteString $
fromMaybe fromMaybe
(joinPath y (approot y) ps (joinPath y ar ps
$ params ++ params') $ params ++ params')
(urlRenderOverride y url) (urlRenderOverride y url)
where where
(ps, params') = renderRoute url (ps, params') = renderRoute url
resolveApproot :: Yesod master => master -> W.Request -> ResolvedApproot
resolveApproot master req =
case approot of
ApprootRelative -> ""
ApprootStatic t -> t
ApprootMaster f -> f master
ApprootRequest f -> f master req

View File

@ -21,7 +21,7 @@ key2 = $(mkCacheKey)
mkYesod "C" [parseRoutes|/ RootR GET|] mkYesod "C" [parseRoutes|/ RootR GET|]
instance Yesod C where approot _ = "" instance Yesod C
getRootR :: Handler () getRootR :: Handler ()
getRootR = do getRootR = do

View File

@ -41,7 +41,7 @@ mkYesod "Y" [parseRoutes|
|] |]
instance Yesod Y where instance Yesod Y where
approot _ = "http://test" approot = ApprootStatic "http://test"
cleanPath _ s@("subsite":_) = Right s cleanPath _ s@("subsite":_) = Right s
cleanPath _ ["bar", ""] = Right ["bar"] cleanPath _ ["bar", ""] = Right ["bar"]
cleanPath _ ["bar"] = Left ["bar", ""] cleanPath _ ["bar"] = Left ["bar", ""]

View File

@ -21,7 +21,7 @@ mkYesod "App" [parseRoutes|
/after_runRequestBody AfterRunRequestBodyR POST /after_runRequestBody AfterRunRequestBodyR POST
|] |]
instance Yesod App where approot _ = "" instance Yesod App
getHomeR :: Handler RepHtml getHomeR :: Handler RepHtml
getHomeR = defaultLayout $ toWidget [hamlet| getHomeR = defaultLayout $ toWidget [hamlet|

View File

@ -18,7 +18,7 @@ mkYesod "Y" [parseRoutes|
|] |]
instance Yesod Y where instance Yesod Y where
approot _ = "http://test" approot = ApprootStatic "http://test"
errorHandler (InternalError e) = return $ chooseRep $ RepPlain $ toContent e errorHandler (InternalError e) = return $ chooseRep $ RepPlain $ toContent e
errorHandler x = defaultErrorHandler x errorHandler x = defaultErrorHandler x

View File

@ -15,8 +15,7 @@ mkYesod "Y" [parseRoutes|
/ RootR GET / RootR GET
|] |]
instance Yesod Y where instance Yesod Y
approot _ = ""
getRootR :: Handler RepHtml getRootR :: Handler RepHtml
getRootR = defaultLayout $ addHamlet [hamlet|<a href=@{RootR}>|] getRootR = defaultLayout $ addHamlet [hamlet|<a href=@{RootR}>|]

View File

@ -15,7 +15,6 @@ import YesodCoreTest.MediaData
mkYesodDispatch "Y" resourcesY mkYesodDispatch "Y" resourcesY
instance Yesod Y where instance Yesod Y where
approot _ = ""
addStaticContent _ _ content = do addStaticContent _ _ content = do
tm <- getRouteToMaster tm <- getRouteToMaster
route <- getCurrentRoute route <- getCurrentRoute

View File

@ -8,7 +8,6 @@ import Test.Hspec.HUnit ()
import Yesod.Core hiding (Request) import Yesod.Core hiding (Request)
import Network.Wai.Test import Network.Wai.Test
import Data.Monoid (mempty) import Data.Monoid (mempty)
import Data.String (fromString)
data Subsite = Subsite data Subsite = Subsite
@ -29,8 +28,7 @@ mkYesod "Y" [parseRoutes|
/subsite SubsiteR Subsite getSubsite /subsite SubsiteR Subsite getSubsite
|] |]
instance Yesod Y where instance Yesod Y
approot _ = fromString ""
getRootR :: Handler () getRootR :: Handler ()
getRootR = return () getRootR = return ()

View File

@ -13,7 +13,7 @@ mkYesod "Y" [parseRoutes|
/r307 R307 GET /r307 R307 GET
/rregular RRegular GET /rregular RRegular GET
|] |]
instance Yesod Y where approot _ = "http://test" instance Yesod Y where approot = ApprootStatic "http://test"
app :: Session () -> IO () app :: Session () -> IO ()
app = yesod Y app = yesod Y

View File

@ -28,7 +28,7 @@ mkYesod "Y" [parseRoutes|
|] |]
instance Yesod Y where instance Yesod Y where
approot _ = "http://test" approot = ApprootStatic "http://test"
getRootR :: Handler RepHtml getRootR :: Handler RepHtml
getRootR = defaultLayout $ toWidgetBody [julius|<not escaped>|] getRootR = defaultLayout $ toWidgetBody [julius|<not escaped>|]

View File

@ -3,6 +3,7 @@ module YesodCoreTest.YesodTest
( yesod ( yesod
, parseRoutes, mkYesod, yesodDispatch, renderRoute, Yesod(..) , parseRoutes, mkYesod, yesodDispatch, renderRoute, Yesod(..)
, redirect , redirect
, Approot (..)
, module Network.Wai , module Network.Wai
, module Network.Wai.Test , module Network.Wai.Test
, module Test.Hspec , module Test.Hspec

View File

@ -83,7 +83,7 @@ type Form x = Html -> MForm ~sitearg~ ~sitearg~ (FormResult x, Widget)
-- Please see the documentation for the Yesod typeclass. There are a number -- Please see the documentation for the Yesod typeclass. There are a number
-- of settings which can be configured by overriding methods here. -- of settings which can be configured by overriding methods here.
instance Yesod ~sitearg~ where instance Yesod ~sitearg~ where
approot = appRoot . settings approot = ApprootMaster $ appRoot . settings
-- Place the session key file in the config folder -- Place the session key file in the config folder
encryptKey _ = fmap Just $ getKey "config/client_session_key.aes" encryptKey _ = fmap Just $ getKey "config/client_session_key.aes"

View File

@ -60,7 +60,7 @@ mkYesodData "~sitearg~" $(parseRoutesFile "config/routes")
-- Please see the documentation for the Yesod typeclass. There are a number -- Please see the documentation for the Yesod typeclass. There are a number
-- of settings which can be configured by overriding methods here. -- of settings which can be configured by overriding methods here.
instance Yesod ~sitearg~ where instance Yesod ~sitearg~ where
approot = appRoot . settings approot = ApprootMaster $ appRoot . settings
-- Place the session key file in the config folder -- Place the session key file in the config folder
encryptKey _ = fmap Just $ getKey "config/client_session_key.aes" encryptKey _ = fmap Just $ getKey "config/client_session_key.aes"