Skip to content

Commit 86f62c1

Browse files
committed
refactor: Use HttpURLConnection constants in alive/spec handlers
1 parent 30e23e8 commit 86f62c1

2 files changed

Lines changed: 6 additions & 3 deletions

File tree

src/main/java/com/retailsvc/http/Handlers.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import static java.net.HttpURLConnection.HTTP_BAD_REQUEST;
55
import static java.net.HttpURLConnection.HTTP_INTERNAL_ERROR;
66
import static java.net.HttpURLConnection.HTTP_NOT_FOUND;
7+
import static java.net.HttpURLConnection.HTTP_NO_CONTENT;
78
import static java.nio.charset.StandardCharsets.UTF_8;
89

910
import com.retailsvc.http.internal.ClasspathResourceHandler;
@@ -61,7 +62,7 @@ public static HttpHandler aliveHandler() {
6162
return new MethodLimitedHandler(
6263
exchange -> {
6364
try (exchange) {
64-
exchange.sendResponseHeaders(204, -1);
65+
exchange.sendResponseHeaders(HTTP_NO_CONTENT, -1);
6566
}
6667
});
6768
}

src/main/java/com/retailsvc/http/internal/ClasspathResourceHandler.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.retailsvc.http.internal;
22

3+
import static java.net.HttpURLConnection.HTTP_OK;
4+
35
import com.sun.net.httpserver.HttpExchange;
46
import com.sun.net.httpserver.HttpHandler;
57
import java.io.IOException;
@@ -34,10 +36,10 @@ public void handle(HttpExchange exchange) throws IOException {
3436
exchange.getResponseHeaders().add("Content-Type", contentType);
3537
if ("HEAD".equals(exchange.getRequestMethod())) {
3638
exchange.getResponseHeaders().add("Content-Length", String.valueOf(bytes.length));
37-
exchange.sendResponseHeaders(200, -1);
39+
exchange.sendResponseHeaders(HTTP_OK, -1);
3840
return;
3941
}
40-
exchange.sendResponseHeaders(200, bytes.length);
42+
exchange.sendResponseHeaders(HTTP_OK, bytes.length);
4143
exchange.getResponseBody().write(bytes);
4244
}
4345
}

0 commit comments

Comments
 (0)