Added support for --setup flag to reset database

auth/auth.go

@@ -53,6 +53,8 @@ func Login(r *http.Request) (bool, error) {
 	return true, nil
 }
 
+// GetUserById returns the user that the given id corresponds to. If no user is found, an
+// error is thrown.
 func GetUserById(id int) (models.User, error) {
 	u := models.User{}
 	stmt, err := db.Conn.Prepare("SELECT id, username, apikey FROM Users WHERE id=?")
@@ -61,12 +63,13 @@ func GetUserById(id int) (models.User, error) {
 	}
 	err = stmt.QueryRow(id).Scan(&u.Id, &u.Username, &u.APIKey)
 	if err != nil {
-		//Return false, but don't return an error
 		return u, err
 	}
 	return u, nil
 }
 
+// GetUserByAPIKey returns the user that the given API Key corresponds to. If no user is found, an
+// error is thrown.
 func GetUserByAPIKey(key []byte) (models.User, error) {
 	u := models.User{}
 	stmt, err := db.Conn.Prepare("SELECT id, username, apikey FROM Users WHERE apikey=?")
@@ -75,7 +78,6 @@ func GetUserByAPIKey(key []byte) (models.User, error) {
 	}
 	err = stmt.QueryRow(key).Scan(&u.Id, &u.Username, &u.APIKey)
 	if err != nil {
-		//Return false, but don't return an error
 		return u, err
 	}
 	return u, nil

controllers/route.go

@@ -83,8 +83,7 @@ func Base(w http.ResponseWriter, r *http.Request) {
 		User    models.User
 		Title   string
 		Flashes []interface{}
-	}{}
-	params.User = ctx.Get(r, "user").(models.User)
+	}{Title: "Dashboard", User: ctx.Get(r, "user").(models.User)}
 	fmt.Println(params.User.Username)
 	getTemplate(w, "dashboard").ExecuteTemplate(w, "base", nil)
 }
@@ -112,9 +111,8 @@ func Login(w http.ResponseWriter, r *http.Request) {
 		User    models.User
 		Title   string
 		Flashes []interface{}
-	}{}
+	}{Title: "Login"}
 	session := ctx.Get(r, "session").(*sessions.Session)
-	params.Title = "Login"
 	switch {
 	case r.Method == "GET":
 		getTemplate(w, "login").ExecuteTemplate(w, "base", params)

db/db.go

@@ -13,22 +13,23 @@ var Conn *sql.DB
 
 // Setup initializes the Conn object
 // It also populates the Gophish Config object
-func Setup() error {
-	createTablesSQL := []string{
-		//Create tables
-		`CREATE TABLE Users (id INTEGER PRIMARY KEY AUTOINCREMENT, username TEXT NOT NULL, hash VARCHAR(60) NOT NULL, apikey VARCHAR(32));`,
-		`CREATE TABLE Campaigns (id INTEGER PRIMARY KEY AUTOINCREMENT, name TEXT NOT NULL, created_date TEXT NOT NULL, completed_date TEXT, status TEXT NOT NULL);`,
-	}
+func Setup(reset bool) error {
 	//If the file already exists, delete it and recreate it
 	_, err := os.Stat(config.Conf.DBPath)
 	if err == nil {
 		os.Remove(config.Conf.DBPath)
 	}
-	fmt.Println("Creating db at " + config.Conf.DBPath)
 	Conn, err = sql.Open("sqlite3", config.Conf.DBPath)
 	if err != nil {
 		return err
 	}
+	if reset {
+		createTablesSQL := []string{
+			//Create tables
+			`CREATE TABLE Users (id INTEGER PRIMARY KEY AUTOINCREMENT, username TEXT NOT NULL, hash VARCHAR(60) NOT NULL, apikey VARCHAR(32));`,
+			`CREATE TABLE Campaigns (id INTEGER PRIMARY KEY AUTOINCREMENT, name TEXT NOT NULL, created_date TEXT NOT NULL, completed_date TEXT, status TEXT NOT NULL);`,
+		}
+		fmt.Println("Creating db at " + config.Conf.DBPath)
 		//Create the tables needed
 		for _, stmt := range createTablesSQL {
 			_, err = Conn.Exec(stmt)
@@ -46,5 +47,6 @@ func Setup() error {
 		if err != nil {
 			return err
 		}
+	}
 	return nil
 }

gophish.go

@@ -40,7 +40,8 @@ var setupFlag = flag.Bool("setup", false, "Starts the initial setup process for
 
 func main() {
 	//Setup the global variables and settings
-	err := db.Setup()
+	flag.Parse()
+	err := db.Setup(*setupFlag)
 	defer db.Conn.Close()
 	if err != nil {
 		fmt.Println(err)