This is an automated email from the ASF dual-hosted git repository.
gavinchou pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 2ad5417a9d0 [fix](be) Use normalized URI for Azure client (#68476)
2ad5417a9d0 is described below
commit 2ad5417a9d0e74a1986f288a6ec781e0cedcb519
Author: Gavin Chou <[email protected]>
AuthorDate: Tue Oct 13 01:01:49 2026 +0800
[fix](be) Use normalized URI for Azure client (#68476)
Related PR: #54291
Problem Summary:
The BE Azure client path computes a normalized container URI, but passes
the original URI to `AzureAuthFactory::create`. If the configured
endpoint produces duplicate path separators, BE can therefore create its
Azure client with a non-canonical path while Recycler uses the
normalized path.
This change passes `normalized_uri` to the client factory so BE and
Recycler use the same Azure container identity.
### Release note
Fix Azure object storage client creation when the configured endpoint
produces duplicate path separators.
---
be/src/util/s3_util.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/be/src/util/s3_util.cpp b/be/src/util/s3_util.cpp
index d42bcae9055..cd5d019a18f 100644
--- a/be/src/util/s3_util.cpp
+++ b/be/src/util/s3_util.cpp
@@ -291,7 +291,7 @@ Result<std::shared_ptr<io::ObjStorageClient>>
S3ClientFactory::_create_azure_cli
VLOG_DEBUG << "uri:" << uri << ", normalized_uri:" << normalized_uri;
std::string tls_debug_context =
build_azure_tls_debug_context(ca_cert_file_path);
- auto built = AzureAuthFactory::create(uri,
+ auto built = AzureAuthFactory::create(normalized_uri,
{
.type =
AzureCredentialType::SHARED_KEY,
.account_name = s3_conf.ak,
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
