Skip to content

protocol: generate temporary table credentials - #107

Draft
littlegrasscao wants to merge 1 commit into
server/split2/16-protocol-table-metadatafrom
server/split2/17-protocol-table-credentials
Draft

littlegrasscao wants to merge 1 commit into
server/split2/16-protocol-table-metadatafrom
server/split2/17-protocol-table-credentials

Conversation

@littlegrasscao

@littlegrasscao littlegrasscao commented Sep 28, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add Delta Sharing POST /shares/{share}/schemas/{schema}/tables/{table}/temporary-table-credentials to OpenSharingProtocolService
  • vend catalog storage credentials, as the share owner, in the protocol credentials JSON (awsTempCredentials / azureUserDelegationSas / gcpOauthToken / r2Credentials) for tables with dir access
  • the body is optional; its location must be the table root or one of its auxiliary locations
  • credential lifetime is the catalog's default (CredentialRequest.ttl null), rather than a new server setting
  • StorageCredentials.covers(location) replaces the prefix check the kernel kept privately
  • a catalog response missing required keys is a catalog error, not a client error

Testing

  • mvn clean install: 159 tests passed
  • ProtocolTableCredentialsTest: placeholder AWS credentials for the table root with no body, explicit root location, schema-grant child, malformed body, location outside the table, ungranted share
  • TemporaryCredentialsTest maps AWS, Azure, GCP, and R2 credentials

Stacked on #106.

@littlegrasscao
littlegrasscao marked this pull request as ready for review September 29, 2026 18:17
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 80beed7 to beb1ff8 Compare September 29, 2026 18:35
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from beb1ff8 to 2389898 Compare September 29, 2026 18:37
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch 5 times, most recently from 6421801 to d237282 Compare October 1, 2026 18:44
@littlegrasscao
littlegrasscao marked this pull request as draft October 7, 2026 01:57
@littlegrasscao littlegrasscao changed the title server: generate temporary table credentials protocol: generate temporary table credentials Oct 9, 2026
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from d237282 to 9751671 Compare October 14, 2026 01:36
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 9751671 to d234d60 Compare October 14, 2026 02:07
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from d234d60 to 4c2ab55 Compare October 14, 2026 19:25
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 4c2ab55 to 0682423 Compare October 15, 2026 16:47
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 0682423 to eeb78ec Compare October 15, 2026 18:14
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from eeb78ec to bdb6621 Compare October 15, 2026 18:29
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from bdb6621 to 54af43d Compare October 15, 2026 23:35
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 54af43d to 7d5dd09 Compare October 15, 2026 23:38
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 7d5dd09 to c9038d7 Compare October 15, 2026 23:54
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from c9038d7 to 86b684e Compare October 16, 2026 00:18
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 86b684e to 602b7f6 Compare October 16, 2026 00:41
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 602b7f6 to 3e72ef7 Compare October 19, 2026 17:13
@zhu-tom
zhu-tom added this pull request to stack #116 October 19, 2026 19:06
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 3e72ef7 to 63c6d19 Compare October 19, 2026 21:44
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 63c6d19 to 6ca73d7 Compare October 19, 2026 22:35
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 6ca73d7 to 24596a8 Compare October 19, 2026 22:37
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch 2 times, most recently from a850517 to 4b6e00c Compare October 21, 2026 17:01
@littlegrasscao
littlegrasscao force-pushed the server/split2/17-protocol-table-credentials branch from 4b6e00c to 6e52ba9 Compare October 21, 2026 19:15
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