[chore] Remove unnecessary deprecated RackLogger override signatures

These turn up as warnings for all usages within juby-rack itself, since most usages use a string. The interface was changed to accept a more generic type, so this is only a breaking change for sub-classes rather than consumers. Since 1.3 has other changes to the logging implementations, we may as well make this required too.

Author: chadlwilson

CHANGELOG.md

@@ -17,6 +17,7 @@ Breaking Java API changes
 - Drop deprecated `org.jruby.rack.RackInput` alias for `org.jruby.rack.ext.Input` class
 - Drop/rename deprecated `RackConfig` and `ServletRackEnvironment` API methods per their earlier comments
 - Drop deprecated `RackLogger` string (`level`) constants
+- Custom `RackLogger` implementations must accept `CharSequence` rather than `String` to allow `RubyString` passthrough 
 
 Breaking Ruby API changes
 - Drop deprecated `JRuby::Rack::RailsFileSystemLayout` alias for `JRuby::Rack::FileSystemLayout`

src/main/java/org/jruby/rack/AbstractRackDispatcher.java

@@ -43,7 +43,6 @@ public void process(RackEnvironment request, RackResponseEnvironment response)
         }
     }
 
-    @SuppressWarnings("deprecation")
     protected void handleException(
             final Exception e,
             final RackEnvironment request,

src/main/java/org/jruby/rack/RackLogger.java

@@ -30,16 +30,6 @@ default void log(CharSequence message, Throwable ex) {
         log(Level.ERROR, message, ex);
     }
 
-    @Deprecated
-    default void log(Level level, String message) {
-        log(level, (CharSequence) message);
-    }
-
-    @Deprecated
-    default void log(Level level, String message, Throwable ex) {
-        log(level, (CharSequence) message, ex);
-    }
-
     abstract class Base implements RackLogger {
         public abstract Level getLevel() ;
         public void setLevel(Level level) { /* noop */ }