From 425a58afd662297a1697349890f8611cf79d2019 Mon Sep 17 00:00:00 2001 From: Unknown Date: Sun, 16 Mar 2014 10:35:25 -0400 Subject: [PATCH] Bug fix --- README.md | 5 +++-- models/user.go | 17 +++++++---------- routers/repo/single.go | 4 +++- routers/user/user.go | 8 ++++---- 4 files changed, 17 insertions(+), 17 deletions(-) diff --git a/README.md b/README.md index 0fb646e5e..f4250a47a 100644 --- a/README.md +++ b/README.md @@ -21,8 +21,9 @@ Please see [Wiki](https://github.com/gogits/gogs/wiki) for project design, devel - SSH protocal support. - Register/delete account. - Create/delete public repository. -- User/repository home page. -- Git repository manipulation. +- User profile page. +- Repository viewer. +- Gravatar support. ## Installation diff --git a/models/user.go b/models/user.go index 4b5a11091..c42599de9 100644 --- a/models/user.go +++ b/models/user.go @@ -168,6 +168,11 @@ func DeleteUser(user *User) error { } } + // Delete user directory. + if err = os.RemoveAll(UserPath(user.Name)); err != nil { + return err + } + _, err = orm.Delete(user) // TODO: delete and update follower information. return err @@ -175,8 +180,8 @@ func DeleteUser(user *User) error { // EncodePasswd encodes password to safe format. func (user *User) EncodePasswd() error { - var err error - user.Passwd, err = EncodePasswd(user.Passwd) + newPasswd, err := scrypt.Key([]byte(user.Passwd), []byte(UserPasswdSalt), 16384, 8, 1, 64) + user.Passwd = fmt.Sprintf("%x", newPasswd) return err } @@ -184,14 +189,6 @@ func UserPath(userName string) string { return filepath.Join(RepoRootPath, userName) } -func EncodePasswd(rawPasswd string) (string, error) { - newPasswd, err := scrypt.Key([]byte(rawPasswd), []byte(UserPasswdSalt), 16384, 8, 1, 64) - if err != nil { - return "", err - } - return fmt.Sprintf("%x", newPasswd), nil -} - func GetUserByKeyId(keyId int64) (*User, error) { user := new(User) has, err := orm.Sql("select a.* from user as a, public_key as b where a.id = b.owner_id and b.id=?", keyId).Get(user) diff --git a/routers/repo/single.go b/routers/repo/single.go index eda30c00a..6bf03ca2b 100644 --- a/routers/repo/single.go +++ b/routers/repo/single.go @@ -46,11 +46,12 @@ func Single(ctx *middleware.Context, params martini.Params) { ctx.Data["Paths"] = Paths ctx.Data["Treenames"] = treenames ctx.Data["IsRepoToolbarSource"] = true + ctx.Data["IsRepositoryOwner"] = strings.ToLower(params["username"]) == ctx.User.LowerName ctx.Data["Files"] = files ctx.Render.HTML(200, "repo/single", ctx.Data) } -func Setting(ctx *middleware.Context) { +func Setting(ctx *middleware.Context, params martini.Params) { if !ctx.Repo.IsValid { return } @@ -62,6 +63,7 @@ func Setting(ctx *middleware.Context) { ctx.Data["Title"] = title + " - settings" ctx.Data["IsRepoToolbarSetting"] = true + ctx.Data["IsRepositoryOwner"] = strings.ToLower(params["username"]) == ctx.User.LowerName ctx.Render.HTML(200, "repo/setting", ctx.Data) } diff --git a/routers/user/user.go b/routers/user/user.go index ad84ff6c7..0ff5058dc 100644 --- a/routers/user/user.go +++ b/routers/user/user.go @@ -157,11 +157,11 @@ func Delete(ctx *middleware.Context) { return } - rawPasswd := ctx.Query("password") - encodedPwd, _ := models.EncodePasswd(rawPasswd) - if len(encodedPwd) == 0 || encodedPwd != ctx.User.Passwd { + tmpUser := models.User{Passwd: ctx.Query("password")} + tmpUser.EncodePasswd() + if len(tmpUser.Passwd) == 0 || tmpUser.Passwd != ctx.User.Passwd { ctx.Data["HasError"] = true - ctx.Data["ErrorMsg"] = "Your password error. Make sure you are owner of this account." + ctx.Data["ErrorMsg"] = "Password is not correct. Make sure you are owner of this account." } else { if err := models.DeleteUser(ctx.User); err != nil { ctx.Data["HasError"] = true