refactor(ws): use more modern fmt.Appendf pattern

Amolith created

Change summary

ws/ws.go | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)

Detailed changes

ws/ws.go 🔗

@@ -137,7 +137,7 @@ func (h Handler) NewHandler(w http.ResponseWriter, r *http.Request) {
 			proj, err := project.GetProject(h.DbConn, proj)
 			if err != nil && !errors.Is(err, sql.ErrNoRows) {
 				w.WriteHeader(http.StatusBadRequest)
-				_, err := w.Write([]byte(fmt.Sprintf("Error getting project: %s", err)))
+				_, err := w.Write(fmt.Appendf(nil, "Error getting project: %s", err))
 				if err != nil {
 					fmt.Println(err)
 				}
@@ -147,7 +147,7 @@ func (h Handler) NewHandler(w http.ResponseWriter, r *http.Request) {
 			proj, err = project.GetReleases(h.DbConn, h.Mu, proj)
 			if err != nil {
 				w.WriteHeader(http.StatusBadRequest)
-				_, err := w.Write([]byte(fmt.Sprintf("Error getting releases: %s", err)))
+				_, err := w.Write(fmt.Appendf(nil, "Error getting releases: %s", err))
 				if err != nil {
 					fmt.Println(err)
 				}
@@ -239,7 +239,7 @@ func (h Handler) LoginHandler(w http.ResponseWriter, r *http.Request) {
 		authorised, err := users.UserAuthorised(h.DbConn, username, password)
 		if err != nil {
 			w.WriteHeader(http.StatusBadRequest)
-			_, err := w.Write([]byte(fmt.Sprintf("Error logging in: %s", err)))
+			_, err := w.Write(fmt.Appendf(nil, "Error logging in: %s", err))
 			if err != nil {
 				fmt.Println(err)
 			}
@@ -258,7 +258,7 @@ func (h Handler) LoginHandler(w http.ResponseWriter, r *http.Request) {
 		session, expiry, err := users.CreateSession(h.DbConn, username)
 		if err != nil {
 			w.WriteHeader(http.StatusBadRequest)
-			_, err := w.Write([]byte(fmt.Sprintf("Error creating session: %s", err)))
+			_, err := w.Write(fmt.Appendf(nil, "Error creating session: %s", err))
 			if err != nil {
 				fmt.Println(err)
 			}
@@ -290,7 +290,7 @@ func (h Handler) LogoutHandler(w http.ResponseWriter, r *http.Request) {
 	err = users.InvalidateSession(h.DbConn, cookie.Value)
 	if err != nil {
 		fmt.Println(err)
-		_, err = w.Write([]byte(fmt.Sprintf("Error logging out: %s", err)))
+		_, err = w.Write(fmt.Appendf(nil, "Error logging out: %s", err))
 		if err != nil {
 			fmt.Println(err)
 		}