aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--cmd/serv.go2
-rwxr-xr-xmodels/db/engine.go4
-rw-r--r--modules/context/api.go4
-rw-r--r--modules/context/context.go6
-rw-r--r--modules/httpcache/httpcache.go2
-rw-r--r--modules/setting/setting.go7
-rw-r--r--modules/templates/base.go2
-rw-r--r--routers/common/middleware.go2
-rw-r--r--routers/install/routes.go4
-rw-r--r--routers/web/base.go4
-rw-r--r--routers/web/web.go2
-rw-r--r--services/auth/sspi_windows.go2
12 files changed, 19 insertions, 22 deletions
diff --git a/cmd/serv.go b/cmd/serv.go
index 2173a3a38b..13c27f1ac0 100644
--- a/cmd/serv.go
+++ b/cmd/serv.go
@@ -80,7 +80,7 @@ func fail(userMessage, logMessage string, args ...interface{}) error {
fmt.Fprintln(os.Stderr, "Gitea:", userMessage)
if len(logMessage) > 0 {
- if !setting.IsProd() {
+ if !setting.IsProd {
fmt.Fprintf(os.Stderr, logMessage+"\n", args...)
}
}
diff --git a/models/db/engine.go b/models/db/engine.go
index 256eb2f3fc..78b4ac22dd 100755
--- a/models/db/engine.go
+++ b/models/db/engine.go
@@ -136,8 +136,8 @@ func NewTestEngine() (err error) {
}
x.SetMapper(names.GonicMapper{})
- x.SetLogger(NewXORMLogger(!setting.IsProd()))
- x.ShowSQL(!setting.IsProd())
+ x.SetLogger(NewXORMLogger(!setting.IsProd))
+ x.ShowSQL(!setting.IsProd)
return syncTables()
}
diff --git a/modules/context/api.go b/modules/context/api.go
index e5216d911f..c978835af8 100644
--- a/modules/context/api.go
+++ b/modules/context/api.go
@@ -95,7 +95,7 @@ func (ctx *APIContext) Error(status int, title string, obj interface{}) {
if status == http.StatusInternalServerError {
log.ErrorWithSkip(1, "%s: %s", title, message)
- if setting.IsProd() && !(ctx.User != nil && ctx.User.IsAdmin) {
+ if setting.IsProd && !(ctx.User != nil && ctx.User.IsAdmin) {
message = ""
}
}
@@ -112,7 +112,7 @@ func (ctx *APIContext) InternalServerError(err error) {
log.ErrorWithSkip(1, "InternalServerError: %v", err)
var message string
- if !setting.IsProd() || (ctx.User != nil && ctx.User.IsAdmin) {
+ if !setting.IsProd || (ctx.User != nil && ctx.User.IsAdmin) {
message = err.Error()
}
diff --git a/modules/context/context.go b/modules/context/context.go
index 0a603cced5..383a69ad63 100644
--- a/modules/context/context.go
+++ b/modules/context/context.go
@@ -225,7 +225,7 @@ func (ctx *Context) NotFound(title string, err error) {
func (ctx *Context) notFoundInternal(title string, err error) {
if err != nil {
log.ErrorWithSkip(2, "%s: %v", title, err)
- if !setting.IsProd() {
+ if !setting.IsProd {
ctx.Data["ErrorMsg"] = err
}
}
@@ -261,7 +261,7 @@ func (ctx *Context) ServerError(title string, err error) {
func (ctx *Context) serverErrorInternal(title string, err error) {
if err != nil {
log.ErrorWithSkip(2, "%s: %v", title, err)
- if !setting.IsProd() {
+ if !setting.IsProd {
ctx.Data["ErrorMsg"] = err
}
}
@@ -645,7 +645,7 @@ func Contexter() func(next http.Handler) http.Handler {
"CurrentURL": setting.AppSubURL + req.URL.RequestURI(),
"PageStartTime": startTime,
"Link": link,
- "IsProd": setting.IsProd(),
+ "IsProd": setting.IsProd,
},
}
// PageData is passed by reference, and it will be rendered to `window.config.pageData` in `head.tmpl` for JavaScript modules
diff --git a/modules/httpcache/httpcache.go b/modules/httpcache/httpcache.go
index 35d4e6dfd8..11b63148d9 100644
--- a/modules/httpcache/httpcache.go
+++ b/modules/httpcache/httpcache.go
@@ -18,7 +18,7 @@ import (
// AddCacheControlToHeader adds suitable cache-control headers to response
func AddCacheControlToHeader(h http.Header, d time.Duration) {
- if setting.IsProd() {
+ if setting.IsProd {
h.Set("Cache-Control", "private, max-age="+strconv.Itoa(int(d.Seconds())))
} else {
h.Set("Cache-Control", "no-store")
diff --git a/modules/setting/setting.go b/modules/setting/setting.go
index 2133184cfc..a1ac090e46 100644
--- a/modules/setting/setting.go
+++ b/modules/setting/setting.go
@@ -419,17 +419,13 @@ var (
PIDFile = "/run/gitea.pid"
WritePIDFile bool
RunMode string
+ IsProd bool
RunUser string
IsWindows bool
HasRobotsTxt bool
InternalToken string // internal access token
)
-// IsProd if it's a production mode
-func IsProd() bool {
- return strings.EqualFold(RunMode, "prod")
-}
-
func getAppPath() (string, error) {
var appPath string
var err error
@@ -906,6 +902,7 @@ func NewContext() {
// Please don't use root as a bandaid to "fix" something that is broken, instead the broken thing should instead be fixed properly.
unsafeAllowRunAsRoot := Cfg.Section("").Key("I_AM_BEING_UNSAFE_RUNNING_AS_ROOT").MustBool(false)
RunMode = Cfg.Section("").Key("RUN_MODE").MustString("prod")
+ IsProd = strings.EqualFold(RunMode, "prod")
// Does not check run user when the install lock is off.
if InstallLock {
currentUser, match := IsRunUserMatchCurrentUser(RunUser)
diff --git a/modules/templates/base.go b/modules/templates/base.go
index cb83143bd3..f753bfbe4e 100644
--- a/modules/templates/base.go
+++ b/modules/templates/base.go
@@ -91,7 +91,7 @@ func HTMLRenderer() *render.Render {
Funcs: NewFuncMap(),
Asset: GetAsset,
AssetNames: GetAssetNames,
- IsDevelopment: !setting.IsProd(),
+ IsDevelopment: !setting.IsProd,
DisableHTTPErrorRendering: true,
})
}
diff --git a/routers/common/middleware.go b/routers/common/middleware.go
index 1d96522dd9..7c5c72f5cc 100644
--- a/routers/common/middleware.go
+++ b/routers/common/middleware.go
@@ -62,7 +62,7 @@ func Middlewares() []func(http.Handler) http.Handler {
if err := recover(); err != nil {
combinedErr := fmt.Sprintf("PANIC: %v\n%s", err, string(log.Stack(2)))
log.Error("%v", combinedErr)
- if setting.IsProd() {
+ if setting.IsProd {
http.Error(resp, http.StatusText(500), 500)
} else {
http.Error(resp, combinedErr, 500)
diff --git a/routers/install/routes.go b/routers/install/routes.go
index e9aca85d8e..ad0003a9e8 100644
--- a/routers/install/routes.go
+++ b/routers/install/routes.go
@@ -40,7 +40,7 @@ func installRecovery() func(next http.Handler) http.Handler {
if err := recover(); err != nil {
combinedErr := fmt.Sprintf("PANIC: %v\n%s", err, string(log.Stack(2)))
log.Error(combinedErr)
- if setting.IsProd() {
+ if setting.IsProd {
http.Error(w, http.StatusText(500), 500)
} else {
http.Error(w, combinedErr, 500)
@@ -63,7 +63,7 @@ func installRecovery() func(next http.Handler) http.Handler {
w.Header().Set(`X-Frame-Options`, setting.CORSConfig.XFrameOptions)
- if !setting.IsProd() {
+ if !setting.IsProd {
store["ErrorMsg"] = combinedErr
}
err = rnd.HTML(w, 500, "status/500", templates.BaseVars().Merge(store))
diff --git a/routers/web/base.go b/routers/web/base.go
index f50c5229b1..16d3192da2 100644
--- a/routers/web/base.go
+++ b/routers/web/base.go
@@ -131,7 +131,7 @@ func Recovery() func(next http.Handler) http.Handler {
sessionStore := session.GetSession(req)
if sessionStore == nil {
- if setting.IsProd() {
+ if setting.IsProd {
http.Error(w, http.StatusText(500), 500)
} else {
http.Error(w, combinedErr, 500)
@@ -164,7 +164,7 @@ func Recovery() func(next http.Handler) http.Handler {
w.Header().Set(`X-Frame-Options`, setting.CORSConfig.XFrameOptions)
- if !setting.IsProd() {
+ if !setting.IsProd {
store["ErrorMsg"] = combinedErr
}
err = rnd.HTML(w, 500, "status/500", templates.BaseVars().Merge(store))
diff --git a/routers/web/web.go b/routers/web/web.go
index caec067671..88565d6d85 100644
--- a/routers/web/web.go
+++ b/routers/web/web.go
@@ -477,7 +477,7 @@ func RegisterRoutes(m *web.Route) {
m.Post("/action/{action}", user.Action)
}, reqSignIn)
- if !setting.IsProd() {
+ if !setting.IsProd {
m.Get("/template/*", dev.TemplatePreview)
}
diff --git a/services/auth/sspi_windows.go b/services/auth/sspi_windows.go
index 346e943988..821a3df459 100644
--- a/services/auth/sspi_windows.go
+++ b/services/auth/sspi_windows.go
@@ -64,7 +64,7 @@ func (s *SSPI) Init() error {
Funcs: templates.NewFuncMap(),
Asset: templates.GetAsset,
AssetNames: templates.GetAssetNames,
- IsDevelopment: !setting.IsProd(),
+ IsDevelopment: !setting.IsProd,
})
return nil
}