fix: Restore the shutdownTimeoutSeconds javadoc

The two content-coding setters were inserted between that javadoc and
the method it documents, so it bound to nothing and shutdownTimeoutSeconds
lost its documentation. Moves it back and folds the cap's two range
checks into the one range they describe.

Also un-nests the ternary that resolving a byte body's content type had
grown, which SonarQube flags as S3358 on new code.

Author: thced

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

@@ -391,24 +391,15 @@ public Builder https(Path certificateChainPem, Path privateKeyPem) {
       return this;
     }
 
-    /**
-     * Sets the default drain timeout used by {@link OpenApiServer#close()}. {@code 0} (the default)
-     * stops immediately; positive values wait up to that many seconds for in-flight exchanges to
-     * finish.
-     */
     /**
      * Ceiling on the inflated size of a gzip request body, 10 MiB by default. A compressed payload
      * can expand by orders of magnitude, so this bounds what a single request may allocate;
      * exceeding it fails the request with 413. Bodies that arrive uncompressed are not affected.
      */
     public Builder maxDecompressedRequestBytes(long maxDecompressedRequestBytes) {
-      if (maxDecompressedRequestBytes <= 0) {
-        throw new IllegalArgumentException(
-            "maxDecompressedRequestBytes must be positive, got " + maxDecompressedRequestBytes);
-      }
-      if (maxDecompressedRequestBytes > Integer.MAX_VALUE) {
+      if (maxDecompressedRequestBytes <= 0 || maxDecompressedRequestBytes > Integer.MAX_VALUE) {
         throw new IllegalArgumentException(
-            "maxDecompressedRequestBytes must not exceed "
+            "maxDecompressedRequestBytes must be between 1 and "
                 + Integer.MAX_VALUE
                 + ", got "
                 + maxDecompressedRequestBytes);
@@ -432,6 +423,11 @@ public Builder minimumGzipResponseBytes(long minimumGzipResponseBytes) {
       return this;
     }
 
+    /**
+     * Sets the default drain timeout used by {@link OpenApiServer#close()}. {@code 0} (the default)
+     * stops immediately; positive values wait up to that many seconds for in-flight exchanges to
+     * finish.
+     */
     public Builder shutdownTimeoutSeconds(int shutdownTimeoutSeconds) {
       if (shutdownTimeoutSeconds < 0) {
         throw new IllegalArgumentException(

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

@@ -129,12 +129,12 @@ private static long declaredLength(Headers headers) {
   private void renderBytes(
       HttpExchange exchange, Headers headers, int status, String contentType, Object body)
       throws IOException {
-    String effectiveContentType =
-        contentType != null ? contentType : (body instanceof byte[] ? OCTET_STREAM : DEFAULT_JSON);
+    String fallback = body instanceof byte[] ? OCTET_STREAM : DEFAULT_JSON;
+    String effectiveContentType = contentType != null ? contentType : fallback;
     byte[] bytes = body instanceof byte[] raw ? raw : serialize(body, effectiveContentType);
     defaultContentType(headers, effectiveContentType);
     byte[] payload = maybeCompress(exchange, headers, status, effectiveContentType, bytes);
-    exchange.sendResponseHeaders(status, payload.length == 0 ? -1 : payload.length);
+    exchange.sendResponseHeaders(status, payload.length == 0 ? UNKNOWN_LENGTH : payload.length);
     if (payload.length > 0) {
       try (OutputStream out = exchange.getResponseBody()) {
         out.write(payload);