cleaned getUser function and fixed issue with generating new uuid

Author: Matus-Dubrava

030_sessions/03_signup/main.go

@@ -1,9 +1,10 @@
 package main
 
 import (
-	"github.com/satori/go.uuid"
 	"html/template"
 	"net/http"
+
+	uuid "github.com/satori/go.uuid"
 )
 
 type user struct {
@@ -30,12 +31,12 @@ func main() {
 }
 
 func index(w http.ResponseWriter, req *http.Request) {
-	u := getUser(w, req)
+	u := getUser(req)
 	tpl.ExecuteTemplate(w, "index.gohtml", u)
 }
 
 func bar(w http.ResponseWriter, req *http.Request) {
-	u := getUser(w, req)
+	u := getUser(req)
 	if !alreadyLoggedIn(req) {
 		http.Redirect(w, req, "/", http.StatusSeeOther)
 		return

030_sessions/03_signup/session.go

@@ -2,26 +2,18 @@ package main
 
 import (
 	"net/http"
-
-	"github.com/satori/go.uuid"
 )
 
-func getUser(w http.ResponseWriter, req *http.Request) user {
+func getUser(req *http.Request) user {
+	var u user
+
 	// get cookie
 	c, err := req.Cookie("session")
 	if err != nil {
-		sID, _ := uuid.NewV4()
-		c = &http.Cookie{
-			Name:  "session",
-			Value: sID.String(),
-		}
-
+		return u
 	}
-	//Next line may not be required, commenting it
-	// http.SetCookie(w, c)
 
 	// if the user exists already, get user
-	var u user
 	if un, ok := dbSessions[c.Value]; ok {
 		u = dbUsers[un]
 	}