-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Open
Description
- Product and API: KMS, SecretManagerClient
- Platform: Java
- Mininum Code:
DefaultProfile profile = DefaultProfile.getProfile(region);
profile.setCloseTrace(true);
SecretManagerClient secretManagerClient =
BaseSecretManagerClientBuilder.standard()
.withCredentialsProvider(
createCredentialsProvider(assumeRoleArn, region))
.withRegion(region)
.build();
return SecretCacheClientBuilder.newCacheClientBuilder(secretManagerClient).build();
I noticed a potential inconsistency in the DefaultProfile class, particularly in lines 70-72:
aliyun-openapi-java-sdk/aliyun-java-sdk-core/src/main/java/com/aliyuncs/profile/DefaultProfile.java
Lines 70 to 72 in 01ed0d0
public synchronized static DefaultProfile getProfile(String regionId) { | |
return new DefaultProfile(regionId); | |
} |
Most of the getProfile methods assign a new DefaultProfile instance to the profile variable, but one method does not follow this pattern. This discrepancy could impact clients relying on this profile, potentially leading to issues like inconsistent trace disabling behavior. Could you clarify if this is the intended behavior or if it might be a bug?
Metadata
Metadata
Assignees
Labels
No labels