[Developer] #424 require explicit ORDS endpoint

Refs #424

src/main/java/com/cloudhandson/vpdbackoffice/domain/probe/ProbeStatus.java

@@ -8,6 +8,8 @@ public enum ProbeStatus {
   OBJECT_DISABLED,
   INVALID_TOKEN,
   OBJECT_NOT_ACCESSIBLE,
+  ORDS_NOT_CONFIGURED,
+  ORDS_UNAVAILABLE,
   ORDS_TIMEOUT,
   INVALID_ORDS_RESPONSE,
   UNKNOWN_ERROR

src/main/java/com/cloudhandson/vpdbackoffice/service/OrdsProbeService.java

@@ -67,6 +67,14 @@ public class OrdsProbeService {
   }
 
   public ProbeResult runProbe(ProbeCommand command) {
+    if (properties.ords().baseUrl() == null || properties.ords().baseUrl().isBlank()) {
+      return auditAndReturn(command, ProbeResult.blocked(
+          ProbeStatus.ORDS_NOT_CONFIGURED,
+          ProbeStatus.ORDS_NOT_CONFIGURED.name(),
+          "ORDS base URL이 설정되지 않았습니다. 실제 ORDS 도메인을 BACKOFFICE_ORDS_BASE_URL에 설정한 뒤 백오피스를 재시작하세요."
+      ));
+    }
+
     BearerTokenRecord token = tokenService.findById(command.keyId());
     if (token == null) {
       return auditAndReturn(command, ProbeResult.blocked(
@@ -102,8 +110,8 @@ public class OrdsProbeService {
       return auditAndReturn(command, ProbeResult.blocked(
           status, status.name(), trimMessage(e.getResponseBodyAsString())));
     } catch (ResourceAccessException e) {
-      ProbeStatus status = errorClassifier.isTimeout(e) ? ProbeStatus.ORDS_TIMEOUT : ProbeStatus.UNKNOWN_ERROR;
-      return auditAndReturn(command, ProbeResult.blocked(status, status.name(), e.getMessage()));
+      ProbeStatus status = classifyResourceAccess(e);
+      return auditAndReturn(command, ProbeResult.blocked(status, status.name(), resourceAccessMessage(status, e)));
     } catch (Exception e) {
       return auditAndReturn(command, ProbeResult.blocked(
           ProbeStatus.INVALID_ORDS_RESPONSE, "INVALID_ORDS_RESPONSE", e.getMessage()));
@@ -193,6 +201,29 @@ public class OrdsProbeService {
     return result;
   }
 
+  private ProbeStatus classifyResourceAccess(ResourceAccessException exception) {
+    if (errorClassifier.isTimeout(exception)) {
+      return ProbeStatus.ORDS_TIMEOUT;
+    }
+    if (errorClassifier.isUnavailable(exception)) {
+      return ProbeStatus.ORDS_UNAVAILABLE;
+    }
+    return ProbeStatus.UNKNOWN_ERROR;
+  }
+
+  private String resourceAccessMessage(ProbeStatus status, ResourceAccessException exception) {
+    String detail = trimMessage(exception.getMessage());
+    if (status == ProbeStatus.ORDS_UNAVAILABLE) {
+      return "ORDS 서버에 연결할 수 없습니다. BACKOFFICE_ORDS_BASE_URL의 실제 ORDS 도메인, ORDS 실행 상태, 네트워크 접근을 확인하세요. 상세: "
+          + detail;
+    }
+    if (status == ProbeStatus.ORDS_TIMEOUT) {
+      return "ORDS 응답 시간이 초과되었습니다. ORDS 상태와 BACKOFFICE_ORDS_TIMEOUT_SECONDS 설정을 확인하세요. 상세: "
+          + detail;
+    }
+    return detail;
+  }
+
   private String trimMessage(String body) {
     if (body == null) {
       return null;

src/main/java/com/cloudhandson/vpdbackoffice/service/ProbeErrorClassifier.java

@@ -1,7 +1,10 @@
 package com.cloudhandson.vpdbackoffice.service;
 
 import com.cloudhandson.vpdbackoffice.domain.probe.ProbeStatus;
+import java.net.ConnectException;
+import java.net.NoRouteToHostException;
 import java.net.SocketTimeoutException;
+import java.net.UnknownHostException;
 import org.springframework.http.HttpStatusCode;
 import org.springframework.stereotype.Component;
 import org.springframework.web.client.ResourceAccessException;
@@ -24,9 +27,19 @@ public class ProbeErrorClassifier {
   }
 
   public boolean isTimeout(ResourceAccessException exception) {
-    Throwable cause = exception;
+    return hasCause(exception, SocketTimeoutException.class);
+  }
+
+  public boolean isUnavailable(ResourceAccessException exception) {
+    return hasCause(exception, ConnectException.class)
+        || hasCause(exception, NoRouteToHostException.class)
+        || hasCause(exception, UnknownHostException.class);
+  }
+
+  private boolean hasCause(Throwable throwable, Class<? extends Throwable> type) {
+    Throwable cause = throwable;
     while (cause != null) {
-      if (cause instanceof SocketTimeoutException) {
+      if (type.isInstance(cause)) {
         return true;
       }
       cause = cause.getCause();

src/main/resources/application.yml

@@ -29,5 +29,5 @@ backoffice:
   token:
     max-days: ${BACKOFFICE_TOKEN_MAX_DAYS:365}
   ords:
-    base-url: ${BACKOFFICE_ORDS_BASE_URL:http://localhost:8081/ords}
+    base-url: ${BACKOFFICE_ORDS_BASE_URL:}
     timeout-seconds: ${BACKOFFICE_ORDS_TIMEOUT_SECONDS:10}