From 91711c1a832bfb996af510f6ebf4c260ab434803 Mon Sep 17 00:00:00 2001
From: takezoe <takezoe@gmail.com>
Date: Sat, 7 Oct 2023 01:32:32 +0900
Subject: [PATCH] Warn if deprecated subclasses of PropertyNamingStrategy is
 used

---
 .../databind/PropertyNamingStrategy.java      | 37 ++++++++++++++++++-
 src/moditect/module-info.java                 |  1 +
 2 files changed, 37 insertions(+), 1 deletion(-)

diff --git a/src/main/java/com/fasterxml/jackson/databind/PropertyNamingStrategy.java b/src/main/java/com/fasterxml/jackson/databind/PropertyNamingStrategy.java
index ae7f514315..ef66d5ab0a 100644
--- a/src/main/java/com/fasterxml/jackson/databind/PropertyNamingStrategy.java
+++ b/src/main/java/com/fasterxml/jackson/databind/PropertyNamingStrategy.java
@@ -5,6 +5,8 @@
 import com.fasterxml.jackson.databind.introspect.AnnotatedMethod;
 import com.fasterxml.jackson.databind.introspect.AnnotatedParameter;
 
+import java.util.logging.Logger;
+
 /**
  * Class that defines how names of JSON properties ("external names")
  * are derived from names of POJO methods and fields ("internal names"),
@@ -318,6 +320,14 @@ public String translate(String input)
     @Deprecated // since 2.12
     public static class UpperCamelCaseStrategy extends PropertyNamingStrategyBase
     {
+        public UpperCamelCaseStrategy() {
+            Logger.getLogger(UpperCamelCaseStrategy.class.getName())
+                .warning(
+                    "PropertyNamingStrategy.UpperCamelCaseStrategy is used but it has been deprecated due to " +
+                    "risk of deadlock. Consider using PropertyNamingStrategies.UpperCamelCaseStrategy instead. " +
+                    "See https://github.com/FasterXML/jackson-databind/issues/2715 for more details.");
+        }
+
         /**
          * Converts camelCase to PascalCase
          *
@@ -353,6 +363,14 @@ public String translate(String input) {
     @Deprecated // since 2.12
     public static class LowerCaseStrategy extends PropertyNamingStrategyBase
     {
+        public LowerCaseStrategy() {
+            Logger.getLogger(LowerCaseStrategy.class.getName())
+                .warning(
+                "PropertyNamingStrategy.LowerCaseStrategy is used but it has been deprecated " +
+                    "due to risk of deadlock. Consider using PropertyNamingStrategies.LowerCaseStrategy instead. " +
+                    "See https://github.com/FasterXML/jackson-databind/issues/2715 for more details.");
+        }
+
         @Override
         public String translate(String input) {
             return input.toLowerCase();
@@ -368,6 +386,14 @@ public String translate(String input) {
     @Deprecated // since 2.12
     public static class KebabCaseStrategy extends PropertyNamingStrategyBase
     {
+        public KebabCaseStrategy() {
+            Logger.getLogger(KebabCaseStrategy.class.getName())
+                .warning(
+                "PropertyNamingStrategy.KebabCaseStrategy is used but it has been deprecated" +
+                    "due to risk of deadlock. Consider using PropertyNamingStrategies.KebabCaseStrategy instead. " +
+                    "See https://github.com/FasterXML/jackson-databind/issues/2715 for more details.");
+        }
+
         @Override
         public String translate(String input) {
             return translateLowerCaseWithSeparator(input, '-');
@@ -381,7 +407,16 @@ public String translate(String input) {
      * for reason for deprecation)
      */
     @Deprecated // since 2.12
-    public static class LowerDotCaseStrategy extends PropertyNamingStrategyBase {
+    public static class LowerDotCaseStrategy extends PropertyNamingStrategyBase
+    {
+        public LowerDotCaseStrategy() {
+            Logger.getLogger(LowerDotCaseStrategy.class.getName())
+                .warning(
+                "PropertyNamingStrategy.LowerDotCaseStrategy is used but it has been deprecated" +
+                    "due to risk of deadlock. Consider using PropertyNamingStrategies.LowerDotCaseStrategy instead. " +
+                    "See https://github.com/FasterXML/jackson-databind/issues/2715 for more details.");
+        }
+
         @Override
         public String translate(String input){
             return translateLowerCaseWithSeparator(input, '.');
diff --git a/src/moditect/module-info.java b/src/moditect/module-info.java
index 9cdded2725..263f40ca15 100644
--- a/src/moditect/module-info.java
+++ b/src/moditect/module-info.java
@@ -1,5 +1,6 @@
 // Generated 08-Mar-2019 using Moditect maven plugin
 module com.fasterxml.jackson.databind {
+    requires java.logging;
     // required for
     // java.beans.ConstructorProperties
     // java.beans.Transient