From 7e6c198f21ba2696954310184f384fee7c277575 Mon Sep 17 00:00:00 2001 From: schaffman5 Date: Mon, 20 Nov 2017 02:48:09 -0500 Subject: [PATCH] Apply LANDING_PAGE config options for logged in users (#2894) * Extended LANDINGPAGE config variable to logged in users and added switch for 'organizations' in addition to 'home' and 'explore'. * Updated comments. Signed-off-by: Mike Schaffer * Extended LANDINGPAGE config variable to trigger when user logs in and added switch for 'organizations' in addition to 'home' and 'explore'. Signed-off-by: Mike Schaffer * Extended LANDINGPAGE config variable to logged in users and added switch for 'organizations' in addition to 'home' and 'explore'. * Updated comments. Signed-off-by: Mike Schaffer * Extended LANDINGPAGE config variable to trigger when user logs in and added switch for 'organizations' in addition to 'home' and 'explore'. Signed-off-by: Mike Schaffer * Removed superfluous conditional and correct whitespace. Signed-off-by: Mike Schaffer * Update app.ini Reverted new line. * Formatting. Signed-off-by: Mike Schaffer --- conf/app.ini | 2 +- modules/setting/setting.go | 7 +++++-- routers/user/auth.go | 2 +- 3 files changed, 7 insertions(+), 4 deletions(-) diff --git a/conf/app.ini b/conf/app.ini index 603f8c14f..7001bb990 100644 --- a/conf/app.ini +++ b/conf/app.ini @@ -164,7 +164,7 @@ STATIC_ROOT_PATH = APP_DATA_PATH = data ; Application level GZIP support ENABLE_GZIP = false -; Landing page for non-logged users, can be "home" or "explore" +; Landing page, can be "home", "explore", or "organizations" LANDING_PAGE = home ; Enables git-lfs support. true or false, default is false. LFS_START_SERVER = false diff --git a/modules/setting/setting.go b/modules/setting/setting.go index df80ebdb5..6fe0214a8 100644 --- a/modules/setting/setting.go +++ b/modules/setting/setting.go @@ -56,8 +56,9 @@ type LandingPage string // enumerates all the landing page types const ( - LandingPageHome LandingPage = "/" - LandingPageExplore LandingPage = "/explore" + LandingPageHome LandingPage = "/" + LandingPageExplore LandingPage = "/explore" + LandingPageOrganizations LandingPage = "/explore/organizations" ) // MarkupParser defines the external parser configured in ini @@ -737,6 +738,8 @@ func NewContext() { switch sec.Key("LANDING_PAGE").MustString("home") { case "explore": LandingPageURL = LandingPageExplore + case "organizations": + LandingPageURL = LandingPageOrganizations default: LandingPageURL = LandingPageHome } diff --git a/routers/user/auth.go b/routers/user/auth.go index eb41e2f99..c353db898 100644 --- a/routers/user/auth.go +++ b/routers/user/auth.go @@ -97,7 +97,7 @@ func checkAutoLogin(ctx *context.Context) bool { ctx.SetCookie("redirect_to", "", -1, setting.AppSubURL) ctx.Redirect(redirectTo) } else { - ctx.Redirect(setting.AppSubURL + "/") + ctx.Redirect(setting.AppSubURL + string(setting.LandingPageURL)) } return true }