diff options
| author | Paolo Borelli <pborelli@gnome.org> | 2015-07-31 08:50:11 +0200 |
|---|---|---|
| committer | Paolo Borelli <pborelli@gnome.org> | 2015-08-15 10:05:12 +0200 |
| commit | 0d38794c7f9b25b2e602cb0d2f686e74cd1ab1cb (patch) | |
| tree | 5ebac231c18bbce2ee1ba5f8e244b26cf198b46d /routers | |
| parent | b35d7eee31e8a33e314d54514854efaeee9d46b1 (diff) | |
| download | gitea-0d38794c7f9b25b2e602cb0d2f686e74cd1ab1cb.tar.gz gitea-0d38794c7f9b25b2e602cb0d2f686e74cd1ab1cb.zip | |
Factor out function to get the current user
The same logic was duplicated in three places. Factor it
out so that we can add further fallbacks in a single place.
Diffstat (limited to 'routers')
| -rw-r--r-- | routers/install.go | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/routers/install.go b/routers/install.go index 3d3e899763..8fed105fc2 100644 --- a/routers/install.go +++ b/routers/install.go @@ -26,6 +26,7 @@ import ( "github.com/gogits/gogs/modules/middleware" "github.com/gogits/gogs/modules/setting" "github.com/gogits/gogs/modules/social" + "github.com/gogits/gogs/modules/user" ) const ( @@ -111,10 +112,7 @@ func Install(ctx *middleware.Context) { // Note(unknwon): it's hard for Windows users change a running user, // so just use current one if config says default. if setting.IsWindows && setting.RunUser == "git" { - form.RunUser = os.Getenv("USER") - if len(form.RunUser) == 0 { - form.RunUser = os.Getenv("USERNAME") - } + form.RunUser = user.CurrentUsername() } else { form.RunUser = setting.RunUser } @@ -201,10 +199,7 @@ func InstallPost(ctx *middleware.Context, form auth.InstallForm) { } // Check run user. - curUser := os.Getenv("USER") - if len(curUser) == 0 { - curUser = os.Getenv("USERNAME") - } + curUser := user.CurrentUsername() if form.RunUser != curUser { ctx.Data["Err_RunUser"] = true ctx.RenderWithErr(ctx.Tr("install.run_user_not_match", form.RunUser, curUser), INSTALL, &form) |