Review comments

dagnir · dagnir · commit 2f95f079e7e8 · 2025-07-02T16:05:34.000-07:00
diff --git a/.changes/next-release/feature-AWSSDKforJavav2-960b6f3.json b/.changes/next-release/feature-AWSSDKforJavav2-960b6f3.json
@@ -0,0 +1,6 @@
+{
+    "type": "feature",
+    "category": "AWS SDK for Java v2",
+    "contributor": "",
+    "description": "Add validation for invalid usages of the `enableEnvironmentBearerToken` codegen customization."
+}
diff --git a/codegen/src/main/java/software/amazon/awssdk/codegen/poet/builder/BaseClientBuilderClass.java b/codegen/src/main/java/software/amazon/awssdk/codegen/poet/builder/BaseClientBuilderClass.java
@@ -327,7 +327,7 @@ private void configureEnvironmentBearerToken(MethodSpec.Builder builder) {
             ValidationEntry entry = ValidationEntry.create(ValidationErrorId.INVALID_CODEGEN_CUSTOMIZATION,
                                                            ValidationErrorSeverity.DANGER,
                                                            "The enableEnvironmentBearerToken customization requires"
-                                                           + " the SRA Auth customization.");
+                                                           + " the useSraAuth customization but it is disabled.");
 
             throw ModelInvalidException.fromEntry(entry);
         }

Original file line number	Diff line number	Diff line change
`@@ -327,7 +327,7 @@ private void configureEnvironmentBearerToken(MethodSpec.Builder builder) {`
`327`	`327`	`ValidationEntry entry = ValidationEntry.create(ValidationErrorId.INVALID_CODEGEN_CUSTOMIZATION,`
`328`	`328`	`ValidationErrorSeverity.DANGER,`
`329`	`329`	`"The enableEnvironmentBearerToken customization requires"`
`330`		`- + " the SRA Auth customization.");`
	`330`	`+ + " the useSraAuth customization but it is disabled.");`
`331`	`331`
`332`	`332`	`throw ModelInvalidException.fromEntry(entry);`
`333`	`333`	`}`