Fix code quality issues (#2118)

controllers/route.go

@@ -301,7 +301,7 @@ func (as *AdminServer) nextOrIndex(w http.ResponseWriter, r *http.Request) {
 			next = path
 		}
 	}
-	http.Redirect(w, r, next, 302)
+	http.Redirect(w, r, next, http.StatusFound)
 }
 
 func (as *AdminServer) handleInvalidLogin(w http.ResponseWriter, r *http.Request, message string) {
@@ -386,7 +386,7 @@ func (as *AdminServer) Login(w http.ResponseWriter, r *http.Request) {
 			as.handleInvalidLogin(w, r, "Invalid Username/Password")
 			return
 		}
-		if u.AccountLocked == true {
+		if u.AccountLocked {
 			as.handleInvalidLogin(w, r, "Account Locked")
 			return
 		}

imap/monitor.go

@@ -174,7 +174,7 @@ func checkForNewEmails(im models.IMAP) {
 					log.Error("Error updating GoPhish email with rid ", rid, ": ", err.Error())
 					continue
 				}
-				if im.DeleteReportedCampaignEmail == true {
+				if im.DeleteReportedCampaignEmail {
 					deleteEmails = append(deleteEmails, m.SeqNum)
 				}
 			}

middleware/ratelimit/ratelimit.go

@@ -91,7 +91,7 @@ func (limiter *PostLimiter) Cleanup() {
 	limiter.Lock()
 	defer limiter.Unlock()
 	for ip, bucket := range limiter.visitors {
-		if time.Now().Sub(bucket.lastSeen) >= limiter.expiry {
+		if time.Since(bucket.lastSeen) >= limiter.expiry {
 			delete(limiter.visitors, ip)
 		}
 	}

models/email_request.go

@@ -144,7 +144,7 @@ func (s *EmailRequest) Generate(msg *gomail.Message) error {
 		log.Error(err)
 	}
 	// don't set the Subject header if it is blank
-	if len(subject) != 0 {
+	if subject != "" {
 		msg.SetHeader("Subject", subject)
 	}
 

models/maillog.go

@@ -215,7 +215,7 @@ func (m *MailLog) Generate(msg *gomail.Message) error {
 		log.Warn(err)
 	}
 	// don't set Subject header if the subject is empty
-	if len(subject) != 0 {
+	if subject != "" {
 		msg.SetHeader("Subject", subject)
 	}