Add breakingChanges to the removal of sigv4 and sigv4a traits (#2567)

Author: qijuew

.changes/next-release/feature-4352fb542ea76e90f06860fb12e80bc88b02616f.json

@@ -0,0 +1,5 @@
+{
+  "type": "feature",
+  "description": "Add breakingChanges to the removal of sigv4 and sigv4a traits",
+  "pull_requests": []
+}

smithy-aws-traits/src/main/resources/META-INF/smithy/aws.auth.smithy

@@ -23,7 +23,17 @@ structure cognitoUserPools {
     traits: [unsignedPayload]
 )
 @externalDocumentation(Reference: "https://docs.aws.amazon.com/general/latest/gr/signature-version-4.html")
-@trait(selector: "service")
+@trait(
+    selector: "service"
+    breakingChanges: [
+        {
+            change: "remove",
+            message: """
+                Removing the existing authentication scheme is not backward compatible \
+                and can break existing clients' authentication."""
+        }
+    ]
+)
 structure sigv4 {
     /// The signature version 4 service signing name to use in the credential
     /// scope when signing requests. This value SHOULD match the `arnNamespace`
@@ -46,7 +56,17 @@ structure sigv4 {
     Reference: "https://docs.aws.amazon.com/general/latest/gr/signature-version-4.html"
     Examples: "https://github.com/aws-samples/sigv4a-signing-examples"
 )
-@trait(selector: "service[trait|aws.auth#sigv4]")
+@trait(
+    selector: "service[trait|aws.auth#sigv4]"
+    breakingChanges: [
+        {
+            change: "remove",
+            message: """
+                Removing the existing authentication scheme is not backward compatible \
+                and can break existing clients' authentication."""
+        }
+    ]
+)
 structure sigv4a {
     /// The signature version 4a service signing name to use in the credential
     /// scope when signing requests. This value SHOULD match the `arnNamespace`

smithy-aws-traits/src/test/resources/software/amazon/smithy/aws/traits/diffs/remove-sigv4.a.smithy

@@ -0,0 +1,22 @@
+$version: "2.0"
+
+metadata suppressions = [
+    {
+        id: "ModifiedTrait.Update.smithy.api#auth"
+        namespace: "ns.foo"
+    }
+]
+
+namespace ns.foo
+
+use aws.auth#sigv4
+
+@auth([sigv4])
+@sigv4(name: "service")
+service Service {
+    operations: [
+        Operation
+    ]
+}
+
+operation Operation {}

smithy-aws-traits/src/test/resources/software/amazon/smithy/aws/traits/diffs/remove-sigv4.b.smithy

@@ -0,0 +1,20 @@
+$version: "2.0"
+
+metadata suppressions = [
+    {
+        id: "ModifiedTrait.Update.smithy.api#auth"
+        namespace: "ns.foo"
+    }
+]
+
+namespace ns.foo
+
+@auth([])
+service Service {
+    operations: [
+        Operation
+    ]
+}
+
+
+operation Operation {}

smithy-aws-traits/src/test/resources/software/amazon/smithy/aws/traits/diffs/remove-sigv4.events

@@ -0,0 +1,7 @@
+[ERROR] ns.foo#Service: Removed trait `aws.auth#sigv4`. Previous trait value: 
+```
+{
+    "name": "service"
+}
+```
+; Removing the existing authentication scheme is not backward compatible and can break existing clients' authentication. | TraitBreakingChange.Remove

smithy-aws-traits/src/test/resources/software/amazon/smithy/aws/traits/diffs/remove-sigv4a.a.smithy

@@ -0,0 +1,24 @@
+$version: "2.0"
+
+metadata suppressions = [
+    {
+        id: "ModifiedTrait.Update.smithy.api#auth"
+        namespace: "ns.foo"
+    }
+]
+
+namespace ns.foo
+
+use aws.auth#sigv4
+use aws.auth#sigv4a
+
+@auth([sigv4, sigv4a])
+@sigv4(name: "service")
+@sigv4a(name: "service")
+service Service {
+    operations: [
+        Operation
+    ]
+}
+
+operation Operation {}

smithy-aws-traits/src/test/resources/software/amazon/smithy/aws/traits/diffs/remove-sigv4a.b.smithy

@@ -0,0 +1,23 @@
+$version: "2.0"
+
+metadata suppressions = [
+    {
+        id: "ModifiedTrait.Update.smithy.api#auth"
+        namespace: "ns.foo"
+    }
+]
+
+namespace ns.foo
+
+use aws.auth#sigv4
+
+@auth([sigv4])
+@sigv4(name: "service")
+service Service {
+    operations: [
+        Operation
+    ]
+}
+
+
+operation Operation {}

smithy-aws-traits/src/test/resources/software/amazon/smithy/aws/traits/diffs/remove-sigv4a.events

@@ -0,0 +1,7 @@
+[ERROR] ns.foo#Service: Removed trait `aws.auth#sigv4a`. Previous trait value: 
+```
+{
+    "name": "service"
+}
+```
+; Removing the existing authentication scheme is not backward compatible and can break existing clients' authentication. | TraitBreakingChange.Remove