From d70f40229eb50974b83a05a7c840dcd9bc3ae81b Mon Sep 17 00:00:00 2001
From: Nick Craig-Wood <nick@craig-wood.com>
Date: Mon, 18 Aug 2025 14:50:14 +0100
Subject: [PATCH] Revert "s3: set useAlreadyExists to false for Alibaba OSS"

This reverts commit 64ed9b175f3f34a99264aab51808ef77f905d4a1.

This fails the integration tests with

s3_internal_test.go:434: Creating a bucket we already have created returned code: No Error
s3_internal_test.go:439:
    	Error Trace:	backend/s3/s3_internal_test.go:439
    	Error:      	Should be true
    	Test:       	TestIntegration/FsMkdir/FsPutFiles/Internal/Versions/Mkdir
    	Messages:   	Need to set UseAlreadyExists quirk
---
 backend/s3/s3.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/backend/s3/s3.go b/backend/s3/s3.go
index de06383af..9b50228fd 100644
--- a/backend/s3/s3.go
+++ b/backend/s3/s3.go
@@ -3628,7 +3628,7 @@ func setQuirks(opt *Options) {
 		useUnsignedPayload = false // AWS has trailer support which means it adds checksums in the trailer without seeking
 	case "Alibaba":
 		useMultipartEtag = false // Alibaba seems to calculate multipart Etags differently from AWS
-		useAlreadyExists = false // returns BucketAlreadyExists
+		useAlreadyExists = true  // returns 200 OK
 	case "HuaweiOBS":
 		// Huawei OBS PFS is not support listObjectV2, and if turn on the urlEncodeListing, marker will not work and keep list same page forever.
 		urlEncodeListings = false