From 64f1b9fb767d6eb7e2d4f14645dd07e4f89a5e63 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=EC=9D=B4=EA=B7=9C=EB=AF=BC?=
 <90364652+20210815@users.noreply.github.com>
Date: Thu, 10 Apr 2025 15:02:54 +0900
Subject: [PATCH 1/2] feat: open Swagger UI automatically when running locally

---
 pom.xml                                       |  2 +-
 .../boot/listeners/SwaggerUiOpener.java       | 66 +++++++++++++++++++
 src/main/resources/application.properties     |  2 +
 3 files changed, 69 insertions(+), 1 deletion(-)
 create mode 100644 src/main/java/ar/com/nanotaboada/java/samples/spring/boot/listeners/SwaggerUiOpener.java

diff --git a/pom.xml b/pom.xml
index 7672f54..a472537 100644
--- a/pom.xml
+++ b/pom.xml
@@ -142,7 +142,7 @@
         <dependency>
             <groupId>org.springdoc</groupId>
             <artifactId>springdoc-openapi-starter-webmvc-ui</artifactId>
-            <version>2.8.6</version>
+            <version>2.2.0</version>
         </dependency>
         <!--
             Spring Boot Starter Actuator
diff --git a/src/main/java/ar/com/nanotaboada/java/samples/spring/boot/listeners/SwaggerUiOpener.java b/src/main/java/ar/com/nanotaboada/java/samples/spring/boot/listeners/SwaggerUiOpener.java
new file mode 100644
index 0000000..e88fb71
--- /dev/null
+++ b/src/main/java/ar/com/nanotaboada/java/samples/spring/boot/listeners/SwaggerUiOpener.java
@@ -0,0 +1,66 @@
+package ar.com.nanotaboada.java.samples.spring.boot.listeners;
+
+import java.awt.Desktop;
+import java.net.URI;
+import java.awt.GraphicsEnvironment;
+
+
+import org.springframework.beans.factory.annotation.Value;
+import org.springframework.boot.context.event.ApplicationReadyEvent;
+import org.springframework.context.ApplicationListener;
+import org.springframework.context.annotation.Profile;
+import org.springframework.stereotype.Component;
+
+@Component
+@Profile("local")
+
+
+public class SwaggerUiOpener implements ApplicationListener<ApplicationReadyEvent>{
+
+    @Value("${server.port}")
+    private int port;
+
+    @Value("${springdoc.swagger-ui.path:/swagger/index.html}")
+    private String swaggerPath;
+
+    @Override
+    public void onApplicationEvent(ApplicationReadyEvent event) {
+        try {
+            String url = "http://localhost:" + port + swaggerPath;
+            System.out.println("✅ Swagger UI will open at: " + url);
+
+            if (Desktop.isDesktopSupported() && !GraphicsEnvironment.isHeadless()) {
+                Desktop.getDesktop().browse(new URI(url));
+            } else {
+                // Fallback: open Swagger UI in browser depending on the OS (Windows, macOS, Linux)
+                String os = System.getProperty("os.name").toLowerCase();
+                Runtime rt = Runtime.getRuntime();
+
+                if (os.contains("win")) {
+                    rt.exec("rundll32 url.dll,FileProtocolHandler " + url);
+                } else if (os.contains("mac")) {
+                    rt.exec("open " + url);
+                } else if (os.contains("nix") || os.contains("nux")) {
+                    String[] browsers = { "xdg-open", "google-chrome", "firefox" };
+                    String browser = null;
+                    for (String b : browsers) {
+                        if (Runtime.getRuntime().exec(new String[] { "which", b }).getInputStream().read() != -1) {
+                            browser = b;
+                            break;
+                        }
+                    }
+                    if (browser != null) {
+                        rt.exec(new String[] { browser, url });
+                    } else {
+                        System.err.println("⚠️ No supported browser found to open Swagger.");
+                    }
+                } else {
+                    System.err.println("⚠️ Unsupported OS for automatic browser launch.");
+                }
+            }
+        } catch (Exception e) {
+            System.err.println("❌ Failed to open Swagger UI: " + e.getMessage());
+        }
+    }
+    
+}
diff --git a/src/main/resources/application.properties b/src/main/resources/application.properties
index d5d7ec3..8c90f98 100644
--- a/src/main/resources/application.properties
+++ b/src/main/resources/application.properties
@@ -13,3 +13,5 @@ info.app.description=Proof of Concept for a RESTful Web Service made with Spring
 
 springdoc.api-docs.path=/docs
 springdoc.swagger-ui.path=/swagger/index.html
+
+spring.profiles.active=local
\ No newline at end of file

From f7e4c190a94c7a0bf5fe167a787671b9980dc9f0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=EC=9D=B4=EA=B7=9C=EB=AF=BC?=
 <90364652+20210815@users.noreply.github.com>
Date: Thu, 10 Apr 2025 15:06:43 +0900
Subject: [PATCH 2/2] refactor: simplify SwaggerUiOpener and resolve SonarCloud
 issues (logging, method size)

---
 .../boot/listeners/SwaggerUiOpener.java       | 69 +++++++++++--------
 1 file changed, 41 insertions(+), 28 deletions(-)

diff --git a/src/main/java/ar/com/nanotaboada/java/samples/spring/boot/listeners/SwaggerUiOpener.java b/src/main/java/ar/com/nanotaboada/java/samples/spring/boot/listeners/SwaggerUiOpener.java
index e88fb71..070407a 100644
--- a/src/main/java/ar/com/nanotaboada/java/samples/spring/boot/listeners/SwaggerUiOpener.java
+++ b/src/main/java/ar/com/nanotaboada/java/samples/spring/boot/listeners/SwaggerUiOpener.java
@@ -2,9 +2,12 @@
 
 import java.awt.Desktop;
 import java.net.URI;
+import java.util.Arrays;
 import java.awt.GraphicsEnvironment;
 
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.boot.context.event.ApplicationReadyEvent;
 import org.springframework.context.ApplicationListener;
@@ -16,8 +19,9 @@
 
 
 public class SwaggerUiOpener implements ApplicationListener<ApplicationReadyEvent>{
-
-    @Value("${server.port}")
+    private static final Logger log = LoggerFactory.getLogger(SwaggerUiOpener.class);
+    
+  @Value("${server.port}")
     private int port;
 
     @Value("${springdoc.swagger-ui.path:/swagger/index.html}")
@@ -25,42 +29,51 @@ public class SwaggerUiOpener implements ApplicationListener<ApplicationReadyEven
 
     @Override
     public void onApplicationEvent(ApplicationReadyEvent event) {
-        try {
-            String url = "http://localhost:" + port + swaggerPath;
-            System.out.println("✅ Swagger UI will open at: " + url);
+        String url = "http://localhost:" + port + swaggerPath;
+        log.info("✅ Swagger UI will open at: {}", url);
 
+        try {
             if (Desktop.isDesktopSupported() && !GraphicsEnvironment.isHeadless()) {
                 Desktop.getDesktop().browse(new URI(url));
             } else {
-                // Fallback: open Swagger UI in browser depending on the OS (Windows, macOS, Linux)
-                String os = System.getProperty("os.name").toLowerCase();
-                Runtime rt = Runtime.getRuntime();
+                launchBrowserByOS(url);
+            }
+        } catch (Exception e) {
+            log.error("❌ Failed to open Swagger UI", e);
+        }
+    }
 
-                if (os.contains("win")) {
-                    rt.exec("rundll32 url.dll,FileProtocolHandler " + url);
-                } else if (os.contains("mac")) {
-                    rt.exec("open " + url);
-                } else if (os.contains("nix") || os.contains("nux")) {
-                    String[] browsers = { "xdg-open", "google-chrome", "firefox" };
-                    String browser = null;
-                    for (String b : browsers) {
-                        if (Runtime.getRuntime().exec(new String[] { "which", b }).getInputStream().read() != -1) {
-                            browser = b;
-                            break;
+    private void launchBrowserByOS(String url) {
+        String os = System.getProperty("os.name").toLowerCase();
+        try {
+            if (os.contains("win")) {
+                new ProcessBuilder("rundll32", "url.dll,FileProtocolHandler", url).start();
+            } else if (os.contains("mac")) {
+                new ProcessBuilder("open", url).start();
+            } else if (os.contains("nix") || os.contains("nux")) {
+                boolean launched = false;
+                for (String browser : Arrays.asList("xdg-open", "google-chrome", "firefox")) {
+                  Process process = new ProcessBuilder("which", browser).start();
+                    try {
+                        int exitCode = process.waitFor();
+                        if (exitCode == 0) {
+                        new ProcessBuilder(browser, url).start();
+                        launched = true;
+                        break;
                         }
+                    } catch (InterruptedException ie) {
+                          Thread.currentThread().interrupt();
+                          log.warn("Browser detection was interrupted", ie);
                     }
-                    if (browser != null) {
-                        rt.exec(new String[] { browser, url });
-                    } else {
-                        System.err.println("⚠️ No supported browser found to open Swagger.");
-                    }
-                } else {
-                    System.err.println("⚠️ Unsupported OS for automatic browser launch.");
                 }
+                if (!launched) {
+                    log.warn("⚠️ No supported browser found to open Swagger.");
+                }
+            } else {
+                log.warn("⚠️ Unsupported OS for automatic browser launch.");
             }
         } catch (Exception e) {
-            System.err.println("❌ Failed to open Swagger UI: " + e.getMessage());
+            log.error("❌ Error while launching browser", e);
         }
     }
-    
 }