localhots/empact
1
0
Fork 0
Code Releases Activity

Don't parse cookies twice

Browse Source
This commit is contained in:
Gregory Eremin 2015-03-07 22:02:15 +07:00
parent 30b57ded50
commit f33a44bb1f
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
server/request.go
View File

@ -28,13 +28,12 @@ type (
)
func parseRequest(w http.ResponseWriter, r *http.Request) (*request, *statRequest) {
setCookie(w, r)
cook, _ := r.Cookie(cookieName)
login, _ := redis.String(redisPool.Get().Do("HGET", "sessions", cook.Value))
sid := sessionID(w, r)
login, _ := redis.String(redisPool.Get().Do("HGET", "sessions", sid))
req := &request{
r: r,
w: w,
sessionID: cook.Value,
sessionID: sid,
login: login,
}
return req, parseStatRequest(r)
@ -77,8 +76,10 @@ func parseStatRequest(r *http.Request) *statRequest {
}
}
func setCookie(w http.ResponseWriter, r *http.Request) {
if cook, err := r.Cookie(cookieName); err != nil {
func sessionID(w http.ResponseWriter, r *http.Request) string {
var cook *http.Cookie
var err error
if cook, err = r.Cookie(cookieName); err != nil {
cook = &http.Cookie{
Name: cookieName,
Value: uuid.New(),
@ -89,4 +90,5 @@ func setCookie(w http.ResponseWriter, r *http.Request) {
http.SetCookie(w, cook)
r.AddCookie(cook)
}
return cook.Value
}