From 93c00e94d1393ed430121dcf7935a89f04287a43 Mon Sep 17 00:00:00 2001 From: Thomas Newton Date: Fri, 13 Dec 2024 16:21:55 +0000 Subject: [PATCH] Adjust comment and increase SAS expiry time --- cpp/src/arrow/filesystem/azurefs.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cpp/src/arrow/filesystem/azurefs.cc b/cpp/src/arrow/filesystem/azurefs.cc index 78f4ad1edd9a9..cc55ff3e87496 100644 --- a/cpp/src/arrow/filesystem/azurefs.cc +++ b/cpp/src/arrow/filesystem/azurefs.cc @@ -416,7 +416,7 @@ Result 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); @@ -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;