Skip to content

Load table credentials from credentials endpoint#3500

Draft
kevinjqliu wants to merge 1 commit into
codex/load-credentials-apifrom
codex/load-credentials-table-integration
Draft

Load table credentials from credentials endpoint#3500
kevinjqliu wants to merge 1 commit into
codex/load-credentials-apifrom
codex/load-credentials-table-integration

Conversation

@kevinjqliu

@kevinjqliu kevinjqliu commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Load table credentials from /credentials during REST table response handling when inline credentials are absent.
  • Only call the endpoint when the catalog advertises loadCredentials and vended-credentials access delegation is requested.
  • Resolve loaded storage credentials by longest prefix against the table location, while keeping inline storage-credentials precedence over catalog config.
  • Avoid the extra /credentials request for remote signing and catalogs that do not advertise the endpoint.

Stacked on #3499.

Testing

  • PYTHONPATH=. pytest tests/catalog/test_rest.py -k 'load_credentials or storage_credentials or load_table_honor_access_delegation or load_table_without_storage_credentials'
  • ruff check pyiceberg/catalog/rest/__init__.py tests/catalog/test_rest.py
  • ruff format --check pyiceberg/catalog/rest/__init__.py tests/catalog/test_rest.py
  • pre-commit hooks during git commit

@kevinjqliu kevinjqliu force-pushed the codex/load-credentials-table-integration branch 14 times, most recently from 31e0af9 to eddce8f Compare June 13, 2026 22:14
@kevinjqliu kevinjqliu force-pushed the codex/load-credentials-table-integration branch from eddce8f to 0f80215 Compare June 13, 2026 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant