fallintoplace opened a new pull request, #1440: URL: https://github.com/apache/iceberg-go/pull/1440
REST configuration was fetched with a session constructed from the initial client options, and that same session remained active after server defaults and overrides were merged. Transport-level configuration therefore appeared in catalog properties without affecting later requests. Initialization now uses the initial session only to fetch `/v1/config`, then creates the catalog session from the merged options. Session construction no longer stores an automatically created OAuth manager in the options, preventing bootstrap authentication state from carrying into the final session. The final client is created before applying the catalog prefix so OAuth endpoints remain rooted at `/v1`. The regression test supplies a required header through config overrides, performs a catalog operation, and verifies that only the post-config request receives it. Tests: - `go test ./...` - `golangci-lint run --timeout=10m` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
