Fix mistake in GetMissingCookieNames

mautrix · Feb 27, 2024 · 9bac58c · 9bac58c
1 parent dfd119f
commit 9bac58c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/messagix/cookies/cookies.go b/messagix/cookies/cookies.go
@@ -90,10 +90,9 @@ func (c *Cookies) GetMissingCookieNames() []MetaCookieName {
 	} else {
 		missingCookies = slices.Clone(IGRequiredCookies)
 	}
-	slices.DeleteFunc(missingCookies, func(name MetaCookieName) bool {
+	return slices.DeleteFunc(missingCookies, func(name MetaCookieName) bool {
 		return c.values[name] != ""
 	})
-	return missingCookies
 }
 
 func (c *Cookies) IsLoggedIn() bool {

diff --git a/provisioning.go b/provisioning.go
@@ -19,6 +19,7 @@ package main
 import (
 	"context"
 	"encoding/json"
+	"fmt"
 	"net/http"
 	_ "net/http/pprof"
 	"strings"
@@ -112,7 +113,7 @@ func (prov *ProvisioningAPI) Login(w http.ResponseWriter, r *http.Request) {
 	missingRequiredCookies := newCookies.GetMissingCookieNames()
 	if len(missingRequiredCookies) > 0 {
 		log.Debug().Any("missing_cookies", missingRequiredCookies).Msg("Missing cookies in login request")
-		jsonResponse(w, http.StatusBadRequest, Error{ErrCode: mautrix.MBadJSON.ErrCode, Error: "Missing some cookies"})
+		jsonResponse(w, http.StatusBadRequest, Error{ErrCode: mautrix.MBadJSON.ErrCode, Error: fmt.Sprintf("Missing cookies: %v", missingRequiredCookies)})
 		return
 	}
 	err = user.Login(ctx, &newCookies)