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 7573d11 commit 93c00e9
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 @@ -416,7 +416,7 @@ Result<std::string> AzureOptions::GenerateSASToken(
if (storage_shared_key_credential_) {
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 @@ -3164,7 +3164,7 @@ class AzureFileSystem::Impl {
std::string sas_token;
{
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 93c00e9

Please sign in to comment.