zymap commented on code in PR #15710:
URL: https://github.com/apache/pulsar/pull/15710#discussion_r880009016
##########
tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/provider/JCloudBlobStoreProvider.java:
##########
@@ -409,14 +428,21 @@ public String getAWSSecretKey() {
}
};
- static final CredentialBuilder ALIYUN_OSS_CREDENTIAL_BUILDER =
(TieredStorageConfiguration config) -> {
- String accountName = System.getenv("ALIYUN_OSS_ACCESS_KEY_ID");
- if (StringUtils.isEmpty(accountName)) {
- throw new IllegalArgumentException("Couldn't get the aliyun oss
access key id.");
+ static final CredentialBuilder S3_CREDENTIAL_BUILDER =
(TieredStorageConfiguration config) -> {
+ String accountName = System.getenv("ACCESS_KEY_ID");
+ // For forward compatibility
+ if (StringUtils.isEmpty(accountName.trim())) {
Review Comment:
StringUtils.isEmpty will check null value
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
