You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by "chibenwa (via GitHub)" <gi...@apache.org> on 2023/07/17 14:34:47 UTC

[GitHub] [james-project] chibenwa commented on a diff in pull request #1646: JAMES-3755 Healthcheck for Rspamd

chibenwa commented on code in PR #1646:
URL: https://github.com/apache/james-project/pull/1646#discussion_r1265461481


##########
third-party/rspamd/src/main/java/org/apache/james/rspamd/client/RspamdHttpClient.java:
##########
@@ -104,9 +104,10 @@ public final int hashCode() {
     public static final String CHECK_V2_ENDPOINT = "/checkV2";
     public static final String LEARN_SPAM_ENDPOINT = "/learnspam";
     public static final String LEARN_HAM_ENDPOINT = "/learnham";
+    public static final String PING_ENDPOINT = "/ping";
     private static final int OK = 200;
     private static final int NO_CONTENT = 204;
-    private static final int FORBIDDEN = 403;
+    private static final int UNAUTHORIZED = 401;

Review Comment:
   We can't reuse an existing HTTP constant for this?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org