fix: store debugEnabled in logger.go instead of supertokens struct

Author: IamMayankThakur

recipe/session/logger_test.go

@@ -51,14 +51,7 @@ func TestLogDebugMessageWhenDebugTrue(t *testing.T) {
 		t.Error(err.Error())
 	}
 
-	supertokensInstance, err := supertokens.GetInstanceOrThrowError()
-
-	if err != nil {
-		t.Error(err.Error())
-	}
-
 	supertokens.LogDebugMessage(logMessage)
-	assert.Equal(t, true, supertokensInstance.Debug)
 	assert.Contains(t, buf.String(), logMessage, "checking log message in logs")
 }
 
@@ -95,14 +88,7 @@ func TestLogDebugMessageWhenDebugFalse(t *testing.T) {
 		t.Error(err.Error())
 	}
 
-	supertokensInstance, err := supertokens.GetInstanceOrThrowError()
-
-	if err != nil {
-		t.Error(err.Error())
-	}
-
 	supertokens.LogDebugMessage(logMessage)
-	assert.Equal(t, false, supertokensInstance.Debug)
 	assert.NotContains(t, buf.String(), logMessage, "checking log message in logs")
 }
 
@@ -138,14 +124,7 @@ func TestLogDebugMessageWhenDebugNotSet(t *testing.T) {
 		t.Error(err.Error())
 	}
 
-	supertokensInstance, err := supertokens.GetInstanceOrThrowError()
-
-	if err != nil {
-		t.Error(err.Error())
-	}
-
 	supertokens.LogDebugMessage(logMessage)
-	assert.Equal(t, false, supertokensInstance.Debug)
 	assert.NotContains(t, buf.String(), logMessage, "checking log message in logs")
 }
 

supertokens/logger.go

@@ -16,7 +16,8 @@ const supertokens_namespace = "com.supertokens"
 */
 
 var (
-	Logger = log.New(os.Stdout, supertokens_namespace, 0)
+	Logger       = log.New(os.Stdout, supertokens_namespace, 0)
+	debugEnabled = false
 )
 
 func formatMessage(message string) string {
@@ -26,7 +27,7 @@ func formatMessage(message string) string {
 
 func LogDebugMessage(message string) {
 	_, exists := os.LookupEnv("SUPERTOKENS_DEBUG")
-	if exists || superTokensInstance.Debug == true {
+	if exists || debugEnabled == true {
 		Logger.Printf(formatMessage(message))
 	}
 }

supertokens/supertokens.go

@@ -35,7 +35,6 @@ type superTokens struct {
 	RecipeModules         []RecipeModule
 	OnSuperTokensAPIError func(err error, req *http.Request, res http.ResponseWriter)
 	Telemetry             *bool
-	Debug                 bool
 }
 
 // this will be set to true if this is used in a test app environment
@@ -55,8 +54,7 @@ func supertokensInit(config TypeInput) error {
 		superTokens.OnSuperTokensAPIError = config.OnSuperTokensAPIError
 	}
 
-	superTokens.Debug = config.Debug
-	superTokensInstance = superTokens
+	debugEnabled = config.Debug
 
 	LogDebugMessage("Started SuperTokens with debug logging (supertokens.Init called)")