You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opennlp.apache.org by rz...@apache.org on 2025/01/17 12:58:27 UTC
(opennlp) branch check-for-http-status-codes created (now 8959737e)
This is an automated email from the ASF dual-hosted git repository.
rzo1 pushed a change to branch check-for-http-status-codes
in repository https://gitbox.apache.org/repos/asf/opennlp.git
at 8959737e OPENNLP-1697 - GH action fail because of 403 returned by sourceforge
This branch includes the following new commits:
new 8959737e OPENNLP-1697 - GH action fail because of 403 returned by sourceforge
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
(opennlp) 01/01: OPENNLP-1697 - GH action fail because of 403 returned by sourceforge
Posted by rz...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
rzo1 pushed a commit to branch check-for-http-status-codes
in repository https://gitbox.apache.org/repos/asf/opennlp.git
commit 8959737ec6b32ba3a2a316a2602a4728f9baacb8
Author: Richard Zowalla <rz...@apache.org>
AuthorDate: Fri Jan 17 13:55:32 2025 +0100
OPENNLP-1697 - GH action fail because of 403 returned by sourceforge
---
.../opennlp/tools/EnabledWhenCDNAvailable.java | 30 ++++++++++++++++++----
1 file changed, 25 insertions(+), 5 deletions(-)
diff --git a/opennlp-tools/src/test/java/opennlp/tools/EnabledWhenCDNAvailable.java b/opennlp-tools/src/test/java/opennlp/tools/EnabledWhenCDNAvailable.java
index ccba1b5d..fccc5526 100644
--- a/opennlp-tools/src/test/java/opennlp/tools/EnabledWhenCDNAvailable.java
+++ b/opennlp-tools/src/test/java/opennlp/tools/EnabledWhenCDNAvailable.java
@@ -22,6 +22,8 @@ import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.net.InetSocketAddress;
import java.net.Socket;
+import java.net.URL;
+import javax.net.ssl.HttpsURLConnection;
import org.junit.jupiter.api.extension.ConditionEvaluationResult;
import org.junit.jupiter.api.extension.ExecutionCondition;
@@ -51,14 +53,32 @@ public @interface EnabledWhenCDNAvailable {
final EnabledWhenCDNAvailable annotation = optional.get();
final String host = annotation.hostname();
try (Socket socket = new Socket()) {
- socket.connect(new InetSocketAddress(host, 80), TIMEOUT_MS);
- return ConditionEvaluationResult.enabled("Resource (CDN) reachable.");
+ // First, try to establish a socket connection to ensure the host is reachable
+ socket.connect(new InetSocketAddress(host, 443), TIMEOUT_MS);
+
+ // Then, try to check the HTTP status by making an HTTPS request
+ final URL url = new URL("https://" + host);
+ final HttpsURLConnection connection = (HttpsURLConnection) url.openConnection();
+ connection.setConnectTimeout(TIMEOUT_MS);
+ connection.setReadTimeout(TIMEOUT_MS);
+ int statusCode = connection.getResponseCode();
+
+ // If the HTTP status code indicates success (2xx range), return enabled
+ if (statusCode >= 200 && statusCode < 300) {
+ return ConditionEvaluationResult.enabled(
+ "Resource (CDN) reachable with status code: " + statusCode);
+ } else {
+ return ConditionEvaluationResult.disabled(
+ "Resource (CDN) reachable, but HTTP status code: " + statusCode);
+
+ }
} catch (IOException e) {
- // Unreachable, unresolvable or timeout
- return ConditionEvaluationResult.disabled("Resource (CDN) unreachable.");
+ return ConditionEvaluationResult.disabled(
+ "Resource (CDN) unreachable.");
}
}
- return ConditionEvaluationResult.enabled("Nothing annotated with DisabledWhenOffline.");
+ return ConditionEvaluationResult.enabled(
+ "Nothing annotated with EnabledWhenCDNAvailable.");
}
}