Merge branch '2.19'

Author: cowtowncoder

release-notes/VERSION-2.x

@@ -62,6 +62,7 @@ Project: jackson-databind
   fails on JDK 17+
 #4997: `ObjectNode` put methods should do null check for key
 #5014: Add `java.lang.Runnable` as unsafe base type in `DefaultBaseTypeLimitingValidator`
+#5020: Support new `@JsonProperty.isRequired` for overridable definition of "required-ness"
 
 2.18.3 (28-Feb-2025)
 

src/main/java/tools/jackson/databind/introspect/JacksonAnnotationIntrospector.java

@@ -419,7 +419,16 @@ public Boolean hasRequiredMarker(MapperConfig<?> config, AnnotatedMember m)
     {
         JsonProperty ann = _findAnnotation(m, JsonProperty.class);
         if (ann != null) {
-            return ann.required();
+            // 11-Mar-2025, tatu: [databind#5020] Support new "isRequired" annotation
+            OptBoolean required = ann.isRequired();
+            if (required != OptBoolean.DEFAULT) {
+                 return required.asBoolean();
+            }
+            // 11-Mar-2025, tatu: Further, in Jackson 3.0 we will now consider legacy
+            //    property ONLY IF SET TO TRUE (i.e. "false" means "use defaults"
+            if (ann.required()) {
+                return Boolean.TRUE;
+            }
         }
         return null;
     }

src/test/java/tools/jackson/databind/deser/creators/RequiredCreatorTest.java

@@ -20,7 +20,7 @@ static class FascistPoint {
 
         @JsonCreator
         public FascistPoint(@JsonProperty(value="x", required=true) Integer x,
-                @JsonProperty(value="y", required=false) Integer y)
+                @JsonProperty(value="y", isRequired=OptBoolean.FALSE) Integer y)
         {
             this.x = x;
             this.y = y;
@@ -34,7 +34,7 @@ static class LoginUserResponse {
         private String userType;
 
         @JsonCreator
-        public LoginUserResponse(@JsonProperty(value = "otp", required = true) String otp,
+        public LoginUserResponse(@JsonProperty(value = "otp", isRequired = OptBoolean.TRUE) String otp,
                 @JsonProperty(value = "userType", required = true) String userType) {
             this.otp = otp;
             this.userType = userType;

src/test/java/tools/jackson/databind/introspect/JacksonAnnotationIntrospectorTest.java

@@ -135,9 +135,9 @@ public String[] findEnumValues(MapperConfig<?> config, AnnotatedClass annotatedC
     }
 
     /*
-    /**********************************************************
-    /* Unit tests
-    /**********************************************************
+    /**********************************************************************
+    /* Test methods
+    /**********************************************************************
      */
 
     /**