Repository navigation
protocol: sign recipient file URLs for cloud storage - #108
Draft
littlegrasscao wants to merge 1 commit into
Draft
littlegrasscao wants to merge 1 commit into
littlegrasscao wants to merge 1 commit into
Conversation
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
2 times, most recently
from
September 29, 2026 18:37
0a73f7d to
27f2865
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
September 29, 2026 21:59
48e5f02 to
168d12b
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
September 29, 2026 22:06
95b8b31 to
dbf6206
Compare
littlegrasscao
marked this pull request as ready for review
September 29, 2026 23:35
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 1, 2026 17:24
42a853b to
8a41489
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 1, 2026 18:34
8a41489 to
218ae45
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 1, 2026 18:39
218ae45 to
9233fb9
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 1, 2026 18:44
9233fb9 to
4e36e66
Compare
littlegrasscao
marked this pull request as draft
October 7, 2026 01:57
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 14, 2026 01:43
4e36e66 to
39841fa
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 14, 2026 02:07
39841fa to
bca6420
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 14, 2026 19:25
bca6420 to
3dad5a4
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 15, 2026 16:47
3dad5a4 to
85ff185
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 15, 2026 18:14
85ff185 to
58daefb
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 15, 2026 18:29
58daefb to
1b7dd09
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 15, 2026 23:35
1b7dd09 to
9fa5e98
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 15, 2026 23:38
9fa5e98 to
03c514f
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 15, 2026 23:54
03c514f to
31beb6a
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 16, 2026 00:18
31beb6a to
576aef6
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 16, 2026 00:41
576aef6 to
9089369
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 19, 2026 17:13
9089369 to
743022d
Compare
zhu-tom
added this pull request to stack #116
October 19, 2026 19:06
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 19, 2026 21:44
743022d to
c414c96
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 19, 2026 22:35
c414c96 to
1567f19
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 19, 2026 22:37
1567f19 to
111a6f7
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
2 times, most recently
from
October 20, 2026 00:31
111a6f7 to
7e61f1a
Compare
littlegrasscao
force-pushed
the
server/split2/18-protocol-query-table
branch
from
October 21, 2026 17:01
7e61f1a to
26ed6ec
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
UrlSignersin core, which picks a signer by path scheme and caps the URL lifetime to the catalog credential expiryOpenSharinguntil the query endpoint uses themregion: the local catalog's FAKE mode returnsus-west-2, STATIC requires it, and ENV requiresAWS_REGIONorAWS_DEFAULT_REGIONTesting
mvn clean install: 171 tests passedS3UrlSignerTest,AzureSasUrlSignerTest,GcsUrlSignerTest,UrlSignersTest: signing offline with placeholder credentials, unknown schemes, and lifetime cappingUrlSignerCloudIntegrationTestsigns a real data file from the S3, Azure, and GCS test tables and reads its first byte; each case is skipped without credentialsLocalCatalogConnectorTestchecks the FAKE-mode regionStacked on #107.