mirror of
https://github.com/Luzifer/cloudkeys-go.git
synced 2024-11-08 14:10:05 +00:00
Knut Ahlers
a1df72edc5
commitf0db1ff1f8
Author: Knut Ahlers <knut@ahlers.me> Date: Sun Dec 24 12:19:56 2017 +0100 Mark option as deprecated Signed-off-by: Knut Ahlers <knut@ahlers.me> commit9891df2a16
Author: Knut Ahlers <knut@ahlers.me> Date: Sun Dec 24 12:11:56 2017 +0100 Fix: Typo Signed-off-by: Knut Ahlers <knut@ahlers.me> commit836006de64
Author: Knut Ahlers <knut@ahlers.me> Date: Sun Dec 24 12:04:20 2017 +0100 Add new dependencies Signed-off-by: Knut Ahlers <knut@ahlers.me> commitd64fee60c8
Author: Knut Ahlers <knut@ahlers.me> Date: Sun Dec 24 11:55:52 2017 +0100 Replace insecure password hashing Prior this commit passwords were hashed with a static salt and using the SHA1 hashing function. This could lead to passwords being attackable in case someone gets access to the raw data stored inside the database. This commit introduces password hashing using bcrypt hashing function which addresses this issue. Old passwords are not automatically re-hashed as they are unknown. Replacing the old password scheme is not that easy and needs #10 to be solved. Therefore the old hashing scheme is kept for compatibility reason. Signed-off-by: Knut Ahlers <knut@ahlers.me> Signed-off-by: Knut Ahlers <knut@ahlers.me> closes #14 closes #15
94 lines
2.5 KiB
Go
94 lines
2.5 KiB
Go
package main
|
|
|
|
import (
|
|
"crypto/sha1"
|
|
"fmt"
|
|
"net/http"
|
|
"strconv"
|
|
"strings"
|
|
|
|
"golang.org/x/crypto/bcrypt"
|
|
|
|
"github.com/flosch/pongo2"
|
|
"github.com/gorilla/mux"
|
|
"github.com/gorilla/sessions"
|
|
)
|
|
|
|
func loginHandler(res http.ResponseWriter, r *http.Request, session *sessions.Session, ctx *pongo2.Context) (*string, error) {
|
|
var (
|
|
username = strings.ToLower(r.FormValue("username"))
|
|
password = r.FormValue("password")
|
|
deprecatedPassword = fmt.Sprintf("%x", sha1.Sum([]byte(cfg.PasswordSalt+r.FormValue("password")))) // Here for backwards compatibility
|
|
)
|
|
|
|
if !storage.IsPresent(createUserFilename(username)) {
|
|
(*ctx)["error"] = true
|
|
return stringPointer("login.html"), nil
|
|
}
|
|
|
|
userFileRaw, err := storage.Read(createUserFilename(username))
|
|
if err != nil {
|
|
fmt.Printf("ERR: Unable to read user file: %s\n", err)
|
|
(*ctx)["error"] = true
|
|
return stringPointer("login.html"), nil
|
|
}
|
|
|
|
userFile, _ := readDataObject(userFileRaw)
|
|
|
|
bcryptValidationError := bcrypt.CompareHashAndPassword([]byte(userFile.MetaData.Password), []byte(password))
|
|
if bcryptValidationError != nil && userFile.MetaData.Password != deprecatedPassword {
|
|
(*ctx)["error"] = true
|
|
return stringPointer("login.html"), nil
|
|
}
|
|
|
|
auth, ok := session.Values["authorizedAccounts"].(authorizedAccounts)
|
|
if !ok {
|
|
auth = authorizedAccounts{}
|
|
}
|
|
|
|
for i, v := range auth {
|
|
if v.Name == username {
|
|
http.Redirect(res, r, fmt.Sprintf("u/%d/overview", i), http.StatusFound)
|
|
return nil, nil
|
|
}
|
|
}
|
|
|
|
auth = append(auth, authorizedAccount{
|
|
Name: username,
|
|
UserFile: createUserFilename(username),
|
|
})
|
|
|
|
session.Values["authorizedAccounts"] = auth
|
|
if err := session.Save(r, res); err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
http.Redirect(res, r, fmt.Sprintf("u/%d/overview", len(auth)-1), http.StatusFound)
|
|
return nil, nil
|
|
}
|
|
|
|
func logoutHandler(res http.ResponseWriter, r *http.Request, session *sessions.Session, ctx *pongo2.Context) (*string, error) {
|
|
session.Values["authorizedAccounts"] = authorizedAccounts{}
|
|
session.Save(r, res)
|
|
http.Redirect(res, r, "overview", http.StatusFound)
|
|
return nil, nil
|
|
}
|
|
|
|
func checkLogin(r *http.Request, session *sessions.Session) (*authorizedAccount, error) {
|
|
vars := mux.Vars(r)
|
|
idx, err := strconv.ParseInt(vars["userIndex"], 10, 64)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
auth, ok := session.Values["authorizedAccounts"].(authorizedAccounts)
|
|
if !ok {
|
|
auth = authorizedAccounts{}
|
|
}
|
|
|
|
if len(auth)-1 < int(idx) {
|
|
return nil, nil
|
|
}
|
|
|
|
return &auth[idx], nil
|
|
}
|