fix(middleware-bucket-endpoint): remove dualstack from hostname before processing

Author: trivikr

packages/middleware-bucket-endpoint/src/bucketHostname.spec.ts

@@ -6,6 +6,7 @@ describe("bucketHostname", () => {
   const region = "us-west-2";
   describe("from bucket name", () => {
     [
+      { baseHostname: "s3.dualstack.us-west-2.amazonaws.com", isCustomEndpoint: false, dualstackEndpoint: true },
       { baseHostname: "s3.us-west-2.amazonaws.com", isCustomEndpoint: false },
       { baseHostname: "beta.example.com", isCustomEndpoint: true },
     ].forEach(({ baseHostname, isCustomEndpoint }) => {

packages/middleware-bucket-endpoint/src/bucketHostname.ts

@@ -51,7 +51,8 @@ const getEndpointFromBucketName = ({
   tlsCompatible = true,
   isCustomEndpoint = false,
 }: BucketHostnameParams): BucketHostname => {
-  const [clientRegion, hostnameSuffix] = isCustomEndpoint ? [region, baseHostname] : getSuffix(baseHostname);
+  const suffixHostname = dualstackEndpoint ? baseHostname.replace(".dualstack", "") : baseHostname;
+  const [clientRegion, hostnameSuffix] = isCustomEndpoint ? [region, baseHostname] : getSuffix(suffixHostname);
   if (pathStyleEndpoint || !isDnsCompatibleBucketName(bucketName) || (tlsCompatible && DOT_PATTERN.test(bucketName))) {
     return {
       bucketEndpoint: false,