Enabled asynchronous token refreshes by default (#1208)

hectorcast-db · web-flow · commit 79e4b3a6e9b0 · 2025-04-17T11:40:08.000Z
## What changes are proposed in this pull request?
Enabled asynchronous token refreshes by default

## How is this tested?
Integration tests will be run using the new default
diff --git a/NEXT_CHANGELOG.md b/NEXT_CHANGELOG.md
@@ -3,6 +3,7 @@
 ## Release v0.64.0
 
 ### New Features and Improvements
+* Enabled asynchronous token refreshes by default ([#1208](https://github.com/databricks/databricks-sdk-go/pull/1208)).
 
 ### Bug Fixes
 
diff --git a/config/experimental/auth/dataplane/dataplane.go b/config/experimental/auth/dataplane/dataplane.go
@@ -28,7 +28,6 @@ func NewEndpointTokenSource(c OAuthClient, cpts auth.TokenSource) *dataPlaneToke
 		client: c,
 		cpts: auth.NewCachedTokenSource(
 			cpts,
-			auth.WithAsyncRefresh(false), // TODO: Enable async refreshes once the feature is stable.
 		),
 	}
 }
@@ -65,7 +64,6 @@ func (dpts *dataPlaneTokenSource) Token(ctx context.Context, endpoint string, au
 			cpts:        dpts.cpts,
 			authDetails: authDetails,
 		},
-		auth.WithAsyncRefresh(false), // TODO: Enable async refresh once the feature is stable.
 	)
 	dpts.sources.Store(key, ts)
 

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,6 @@ func NewEndpointTokenSource(c OAuthClient, cpts auth.TokenSource) *dataPlaneToke`
`28`	`28`	`client: c,`
`29`	`29`	`cpts: auth.NewCachedTokenSource(`
`30`	`30`	`cpts,`
`31`		`- auth.WithAsyncRefresh(false), // TODO: Enable async refreshes once the feature is stable.`
`32`	`31`	`),`
`33`	`32`	`}`
`34`	`33`	`}`
`@@ -65,7 +64,6 @@ func (dpts *dataPlaneTokenSource) Token(ctx context.Context, endpoint string, au`
`65`	`64`	`cpts: dpts.cpts,`
`66`	`65`	`authDetails: authDetails,`
`67`	`66`	`},`
`68`		`- auth.WithAsyncRefresh(false), // TODO: Enable async refresh once the feature is stable.`
`69`	`67`	`)`
`70`	`68`	`dpts.sources.Store(key, ts)`
`71`	`69`