Browse Source

use correct length state keys

Anish Mukherjee 2 years ago
parent
commit
25e227d551
2 changed files with 2 additions and 4 deletions
  1. 1 2
      auth/auth.go
  2. 1 2
      auth/nodesession.go

+ 1 - 2
auth/auth.go

@@ -2,7 +2,6 @@ package auth
 
 import (
 	"encoding/base64"
-	"encoding/hex"
 	"encoding/json"
 	"errors"
 	"fmt"
@@ -183,7 +182,7 @@ func HandleHeadlessSSO(w http.ResponseWriter, r *http.Request) {
 	defer conn.Close()
 
 	req := &netcache.CValue{User: "", Pass: ""}
-	stateStr := hex.EncodeToString([]byte(logic.RandomString(headless_signin_length)))
+	stateStr := logic.RandomString(headless_signin_length)
 	if err = netcache.Set(stateStr, req); err != nil {
 		logger.Log(0, "Failed to process sso request -", err.Error())
 		return

+ 1 - 2
auth/nodesession.go

@@ -1,7 +1,6 @@
 package auth
 
 import (
-	"encoding/hex"
 	"encoding/json"
 	"fmt"
 	"strings"
@@ -45,7 +44,7 @@ func SessionHandler(conn *websocket.Conn) {
 	req.Pass = ""
 	req.User = ""
 	// Add any extra parameter provided in the configuration to the Authorize Endpoint request??
-	stateStr := hex.EncodeToString([]byte(logic.RandomString(node_signin_length)))
+	stateStr := logic.RandomString(node_signin_length)
 	if err := netcache.Set(stateStr, req); err != nil {
 		logger.Log(0, "Failed to process sso request -", err.Error())
 		return