🐛 Fix: remove context deadline from oauth2 tokenSource (#110)

Author: mullerpeter

package.json

@@ -1,7 +1,7 @@
 {
   "name": "mullerpeter-databricks-datasource",
   "private": true,
-  "version": "1.3.4",
+  "version": "1.3.5",
   "description": "Databricks SQL Connector",
   "scripts": {
     "build": "webpack -c ./.config/webpack/webpack.config.ts --env production",

pkg/integrations/oauth2_client_credentials.go

@@ -8,7 +8,6 @@ import (
 	"golang.org/x/oauth2/clientcredentials"
 	"net/http"
 	"sync"
-	"time"
 )
 
 type oauth2ClientCredentials struct {
@@ -26,6 +25,7 @@ func (c *oauth2ClientCredentials) Authenticate(r *http.Request) error {
 	if c.tokenSource != nil {
 		token, err := c.tokenSource.Token()
 		if err != nil {
+			log.DefaultLogger.Error("token fetching failed", "err", err)
 			return err
 		}
 		token.SetAuthHeader(r)
@@ -39,23 +39,17 @@ func (c *oauth2ClientCredentials) Authenticate(r *http.Request) error {
 		Scopes:       c.scopes,
 	}
 
-	// Create context with 1m timeout to cancel token fetching
-	ctx, cancel := context.WithTimeout(context.Background(), 1*time.Minute)
-	if cancel != nil {
-		log.DefaultLogger.Debug("ignoring defer for timeout to not cancel original request")
-	}
-
-	c.tokenSource = config.TokenSource(ctx)
+	c.tokenSource = config.TokenSource(context.Background())
 
 	log.DefaultLogger.Debug("token fetching started")
 	token, err := c.tokenSource.Token()
 
 	if err != nil {
 		log.DefaultLogger.Error("token fetching failed", "err", err)
 		return err
-	} else {
-		log.DefaultLogger.Debug("token fetched successfully")
 	}
+
+	log.DefaultLogger.Debug("token fetched successfully")
 	token.SetAuthHeader(r)
 
 	return nil