This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-net.git

commit 98d6395b60d7431e496245fb718ec5ea8634cf39
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Thu Sep 12 10:32:33 2024 -0400

    Convert cascading if-else to switch
---
 .../commons/net/examples/ftp/FTPClientExample.java   | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git 
a/src/main/java/org/apache/commons/net/examples/ftp/FTPClientExample.java 
b/src/main/java/org/apache/commons/net/examples/ftp/FTPClientExample.java
index 126c9a7c..bd83852c 100644
--- a/src/main/java/org/apache/commons/net/examples/ftp/FTPClientExample.java
+++ b/src/main/java/org/apache/commons/net/examples/ftp/FTPClientExample.java
@@ -255,12 +255,20 @@ public final class FTPClientExample {
                 }
             }
             ftp = ftps;
-            if ("all".equals(trustmgr)) {
-                
ftps.setTrustManager(TrustManagerUtils.getAcceptAllTrustManager());
-            } else if ("valid".equals(trustmgr)) {
-                
ftps.setTrustManager(TrustManagerUtils.getValidateServerCertificateTrustManager());
-            } else if ("none".equals(trustmgr)) {
-                ftps.setTrustManager(null);
+            if (trustmgr != null) {
+                switch (trustmgr) {
+                case "all":
+                    
ftps.setTrustManager(TrustManagerUtils.getAcceptAllTrustManager());
+                    break;
+                case "valid":
+                    
ftps.setTrustManager(TrustManagerUtils.getValidateServerCertificateTrustManager());
+                    break;
+                case "none":
+                    ftps.setTrustManager(null);
+                    break;
+                default:
+                    break;
+                }
             }
         }
 

Reply via email to