Skip to content

Commit

Permalink
Adjust comment and increase SAS expiry time
Browse files Browse the repository at this point in the history
  • Loading branch information
Tom-Newton committed Dec 13, 2024
1 parent 9c6ec0e commit f4177be
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cpp/src/arrow/filesystem/azurefs.cc
Original file line number Diff line number Diff line change
Expand Up @@ -441,7 +441,7 @@ Result<std::string> AzureOptions::GenerateSASToken(
if (credential_kind_ == CredentialKind::kStorageSharedKey) {
return builder->GenerateSasToken(*storage_shared_key_credential_);
} else {
// GH-39344: This part isn't tested. This may not work.
// GH-39344: This part isn't tested.
try {
auto delegation_key_response = client->GetUserDelegationKey(builder->ExpiresOn);
return builder->GenerateSasToken(delegation_key_response.Value, account_name);
Expand Down Expand Up @@ -3193,7 +3193,7 @@ class AzureFileSystem::Impl {
sas_token = "";
} else {
Storage::Sas::BlobSasBuilder builder;
std::chrono::seconds available_period(60);
std::chrono::seconds available_period(600);
builder.ExpiresOn = std::chrono::system_clock::now() + available_period;
builder.BlobContainerName = src.container;
builder.BlobName = src.path;
Expand Down

0 comments on commit f4177be

Please sign in to comment.