Recipes for methods and classes renamed or moved when switching to Java 17

src/main/resources/META-INF/rewrite/java-version-17.yml

@@ -49,7 +49,10 @@ recipeList:
       groupId: com.sonatype.clm
       artifactId: clm-maven-plugin
       newVersion: 2.47.6-01
-
+  - org.openrewrite.java.migrate.RemovedZipFinalizeMethods
+  - org.openrewrite.java.migrate.RemovedSSLSessionGetPeerCertificateChainMethodImpl
+  - org.openrewrite.java.migrate.SunNetSslPackageUnavailable
+  - org.openrewrite.java.migrate.RemovedRMIConnectorServerCredentialTypesConstant
 ---
 type: specs.openrewrite.org/v1beta/recipe
 name: org.openrewrite.java.migrate.JavaVersion17
@@ -122,3 +125,72 @@ recipeList:
   - org.openrewrite.java.ChangeMethodAccessLevel:
       methodPattern: "*..* premain(java.lang.String, java.lang.instrument.Instrumentation)"
       newAccessLevel: public
+---
+type: specs.openrewrite.org/v1beta/recipe
+name: org.openrewrite.java.migrate.RemovedZipFinalizeMethods
+displayName: Remove `finalize` method in `java.util.zip.Zipfile`, `java.util.zip.Inflater` and  `java.util.zip.Deflater`
+description: >
+  The `finalize` methods in `java.util.zip.ZipFile` is replaced with `close` method and is replaced by `end` method in 
+   `java.util.zip.Inflater` and `java.util.zip.Deflater` as it is no longer available in Java SE 12 and later.
+tags:
+  - java17
+recipeList:
+  - org.openrewrite.java.ChangeMethodName:
+      methodPattern: "java.util.zip.Inflater finalize()"
+      newMethodName: end
+      ignoreDefinition: true
+      matchOverrides: true
+  - org.openrewrite.java.ChangeMethodName:
+      methodPattern: "java.util.zip.Deflater finalize()"
+      newMethodName: end
+      ignoreDefinition: true
+  - org.openrewrite.java.ChangeMethodName:
+      methodPattern: "java.util.ZipFile finalize()"
+      newMethodName: close
+      ignoreDefinition: true
+---
+type: specs.openrewrite.org/v1beta/recipe
+name: org.openrewrite.java.migrate.RemovedSSLSessionGetPeerCertificateChainMethodImpl
+displayName: Remove `SSLSession.getPeerCertificateChain()` method
+description: >
+  The `javax.net.ssl.SSLSession.getPeerCertificateChain()` method implementation was removed from the SunJSSE provider and HTTP client implementation in Java SE 15. 
+  The default implementation will now throw an `UnsupportedOperationException`. 
+  Applications using this method should be updated to use the `javax.net.ssl.SSLSession.getPeerCertificates()` method instead.
+tags:
+  - java17
+recipeList:
+  - org.openrewrite.java.ChangeMethodName:
+      methodPattern: "*.net.ssl.SSLSession getPeerCertificateChain()"
+      newMethodName: getPeerCertificates
+      ignoreDefinition: true
+      matchOverrides: true
+---
+type: specs.openrewrite.org/v1beta/recipe
+name: org.openrewrite.java.migrate.SunNetSslPackageUnavailable
+displayName: Remove `com.sun.net.ssl` package
+description: >
+  The internal API `com.sun.net.ssl` is removed. The package was intended for internal use only and replacement APIs can be found in the javax.net.ssl package.
+tags:
+  - java17
+recipeList:
+  - org.openrewrite.java.ChangePackage:
+      oldPackageName: com.sun.net.ssl
+      newPackageName: javax.net.ssl
+---
+type: specs.openrewrite.org/v1beta/recipe
+name: org.openrewrite.java.migrate.RemovedRMIConnectorServerCredentialTypesConstant
+displayName: Remove `RMIConnectorServer.CREDENTIAL_TYPES` constant
+description: >
+  This recipe  replaces `RMIConnectorServer.CREDENTIAL_TYPES` constant with the `RMIConnectorServer.CREDENTIALS_FILTER_PATTERN` constant and a filter pattern
+tags:
+  - java17
+recipeList:
+  - org.openrewrite.java.ReplaceConstantWithAnotherConstant:
+      existingFullyQualifiedConstantName: javax.management.remote.rmi.RMIConnectorServer.CREDENTIAL_TYPES
+      fullyQualifiedConstantName:  javax.management.remote.rmi.RMIConnectorServer.CREDENTIALS_FILTER_PATTERN
+
+
+
+
+
+

src/test/java/org/openrewrite/java/migrate/SunNetSslPackageUnavailableTest.java

@@ -0,0 +1,70 @@
+/*
+ * Copyright 2024 the original author or authors.
+ * <p>
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * <p>
+ * https://www.apache.org/licenses/LICENSE-2.0
+ * <p>
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.openrewrite.java.migrate;
+
+import org.junit.jupiter.api.Test;
+import org.openrewrite.DocumentExample;
+import org.openrewrite.InMemoryExecutionContext;
+import org.openrewrite.config.Environment;
+import org.openrewrite.java.JavaParser;
+import org.openrewrite.java.migrate.jakarta.RemoveBeanIsNullable;
+import org.openrewrite.test.RecipeSpec;
+import org.openrewrite.test.RewriteTest;
+
+import static org.openrewrite.java.Assertions.*;
+
+
+public class SunNetSslPackageUnavailableTest implements RewriteTest {
+
+    @Override
+    public void defaults(RecipeSpec spec) {
+        spec.parser(JavaParser.fromJavaVersion()
+            .classpathFromResources(new InMemoryExecutionContext(),
+              "com.sun.net.ssl"))
+          .recipeFromResource("/META-INF/rewrite/java-version-17.yml", "org.openrewrite.java.migrate.SunNetSslPackageUnavailable");
+    }
+    @Test
+    void sunNetSslPackageUnavailableTest(){
+        rewriteRun(
+            //language=java
+            java(
+              """
+              package testing.stuff;
+
+              import com.sun.net.ssl.HttpsURLConnection;
+
+              public class TestSunNetSsl {
+                    public void useThePackages() {
+                            HttpsURLConnection con; 
+                    }
+              }
+              """,
+              """
+              package testing.stuff;
+
+              import javax.net.ssl.HttpsURLConnection;
+
+              public class TestSunNetSsl {
+                    public void useThePackages() {
+                            HttpsURLConnection con;            
+                    }
+              }
+              """
+            )
+        );
+    }
+
+}

src/test/java/org/openrewrite/java/migrate/UpgradeToJava17Test.java

@@ -17,6 +17,7 @@
 
 import org.junit.jupiter.api.Test;
 import org.openrewrite.DocumentExample;
+import org.openrewrite.InMemoryExecutionContext;
 import org.openrewrite.config.Environment;
 import org.openrewrite.java.marker.JavaVersion;
 import org.openrewrite.test.RecipeSpec;
@@ -291,7 +292,6 @@ void bar(LogRecord record) {
             ), 17)
         );
     }
-
     @Test
     void needToUpgradeMavenCompilerPluginToSupportReleaseTag() {
         rewriteRun(
@@ -527,4 +527,54 @@ void lombokBumpedGoingTo17() {
           )
         );
     }
+
+
+
+    @Test
+    void removedSSLSessionGetPeerCertificateChainMethodImplTest(){
+        rewriteRun(
+          //language=java
+          java(
+            """
+             package com.test;
+
+             import java.security.cert.Certificate;
+
+             import javax.net.ssl.SSLContext;
+             import javax.net.ssl.SSLEngine;
+             import javax.net.ssl.SSLSession;
+
+             public class RemovedSSLSessionGetPeerCertificateChainMethodImplApp {
+
+                     public void test() throws Exception {
+                         SSLEngine sslEngine = SSLContext.getDefault().createSSLEngine();
+                         SSLSession session = sslEngine.getHandshakeSession();
+                         session.getPeerCertificateChain(); //This should trigger
+                         Certificate[] certs = session.getPeerCertificates(); //This should not trigger
+                     }
+             }
+            """,
+            """
+             package com.test;
+
+             import java.security.cert.Certificate;
+
+             import javax.net.ssl.SSLContext;
+             import javax.net.ssl.SSLEngine;
+             import javax.net.ssl.SSLSession;
+
+             public class RemovedSSLSessionGetPeerCertificateChainMethodImplApp {
+
+                     public void test() throws Exception {
+                         SSLEngine sslEngine = SSLContext.getDefault().createSSLEngine();
+                         SSLSession session = sslEngine.getHandshakeSession();
+                         session.getPeerCertificates(); //This should trigger
+                         Certificate[] certs = session.getPeerCertificates(); //This should not trigger
+                     }
+             }
+              """
+          )
+        );
+    }
+
 }