Update S3 image store URL format to use 's3://' scheme

Damans227 · Damans227 · commit f2dfd10178ab · 2026-02-17T11:52:34.000-05:00
diff --git a/api/src/main/java/org/apache/cloudstack/api/command/admin/storage/AddImageStoreS3CMD.java b/api/src/main/java/org/apache/cloudstack/api/command/admin/storage/AddImageStoreS3CMD.java
@@ -131,7 +131,7 @@ public void execute() throws ResourceUnavailableException, InsufficientCapacityE
             dm.put(ApiConstants.S3_USE_TCP_KEEPALIVE, getUseTCPKeepAlive().toString());
         }
 
-        String url = getEndPoint() + "/" + getBucketName();
+        String url = "s3://" + getEndPoint().replaceFirst("^https?://", "") + "/" + getBucketName();
         try{
             ImageStore result = _storageService.discoverImageStore(null, url, "S3", zoneId, dm);
             ImageStoreResponse storeResponse;

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ public void execute() throws ResourceUnavailableException, InsufficientCapacityE`
`131`	`131`	`dm.put(ApiConstants.S3_USE_TCP_KEEPALIVE, getUseTCPKeepAlive().toString());`
`132`	`132`	`}`
`133`	`133`
`134`		`- String url = getEndPoint() + "/" + getBucketName();`
	`134`	`+ String url = "s3://" + getEndPoint().replaceFirst("^https?://", "") + "/" + getBucketName();`
`135`	`135`	`try{`
`136`	`136`	`ImageStore result = _storageService.discoverImageStore(null, url, "S3", zoneId, dm);`
`137`	`137`	`ImageStoreResponse storeResponse;`