Update version.

Allow the default max size for form config to be removed so that it is possible to use a single max body size.

Author: robotdan

README.md

@@ -2,7 +2,7 @@
 
 ### Latest versions
 
-* Latest stable version: `1.2.0`
+* Latest stable version: `1.3.0`
    * Now with 100% more virtual threads!
 * Prior stable version `0.3.7` 
 
@@ -27,20 +27,20 @@ To add this library to your project, you can include this dependency in your Mav
 <dependency>
   <groupId>io.fusionauth</groupId>
   <artifactId>java-http</artifactId>
-  <version>1.2.0</version>
+  <version>1.3.0</version>
 </dependency>
 ```
 
 If you are using Gradle, you can add this to your build file:
 
 ```groovy
-implementation 'io.fusionauth:java-http:1.2.0'
+implementation 'io.fusionauth:java-http:1.3.0'
 ```
 
 If you are using Savant, you can add this to your build file:
 
 ```groovy
-dependency(id: "io.fusionauth:java-http:1.2.0")
+dependency(id: "io.fusionauth:java-http:1.3.0")
 ```
 
 ## Examples Usages:

build.savant

@@ -18,7 +18,7 @@ restifyVersion = "4.2.1"
 slf4jVersion = "2.0.17"
 testngVersion = "7.11.0"
 
-project(group: "io.fusionauth", name: "java-http", version: "1.2.0", licenses: ["ApacheV2_0"]) {
+project(group: "io.fusionauth", name: "java-http", version: "1.3.0", licenses: ["ApacheV2_0"]) {
   workflow {
     fetch {
       // Dependency resolution order:

pom.xml

@@ -2,7 +2,7 @@
   <modelVersion>4.0.0</modelVersion>
   <groupId>io.fusionauth</groupId>
   <artifactId>java-http</artifactId>
-  <version>1.2.0</version>
+  <version>1.3.0</version>
   <packaging>jar</packaging>
 
   <name>Java HTTP library (client and server)</name>

src/main/java/io/fusionauth/http/server/Configurable.java

@@ -201,8 +201,7 @@ default T withMaxPendingSocketConnections(int maxPendingSocketConnections) {
    *   <li>*</li>
    * </ol>
    * <p>
-   * If the provided configuration does not contain the initial default values for "*" and "application/x-www-form-urlencoded", the
-   * server default values will be retained.
+   * If the provided configuration does not contain the initial default identified by the "*" key, the server default value will be retained.
    * key.
    * <p>
    * Set any value to -1 to disable this limitation.

src/main/java/io/fusionauth/http/server/HTTPServerConfiguration.java

@@ -490,9 +490,9 @@ public HTTPServerConfiguration withMaxRequestBodySize(Map<String, Integer> maxRe
       }
     }
 
-    // This preserves the default values in the field definition if the incoming Map does not contain them. Otherwise, they are overridden
     this.maxRequestBodySize.clear();
-    this.maxRequestBodySize.putAll(DefaultMaxRequestSizes);
+    // Add back a default to ensure we always have a fallback, can still be modified by the incoming configuration.
+    this.maxRequestBodySize.put("*", DefaultMaxRequestSizes.get("*"));
     this.maxRequestBodySize.putAll(maxRequestBodySize);
     return this;
   }

src/test/java/io/fusionauth/http/FormDataTest.java

@@ -74,7 +74,7 @@ public void post_server_configuration_max_form_data(String scheme, boolean chunk
     // Exceeded
     withScheme(scheme)
         .withChunked(chunked)
-        .withBodyParameterCount(42 * 1024)  // 43,008
+        .withBodyParameterCount(42 * 1024)
         .withBodyParameterSize(128)
         // 4k * 33 > 128k
         .withConfiguration(config -> config.withMaxRequestBodySize(Map.of(ContentTypes.Form, 128 * 1024)))
@@ -89,7 +89,7 @@ public void post_server_configuration_max_form_data(String scheme, boolean chunk
     // Large, but max size has been disabled
     withScheme(scheme)
         .withChunked(chunked)
-        .withBodyParameterCount(42 * 1024)  // 43,008
+        .withBodyParameterCount(42 * 1024)
         .withBodyParameterSize(128)
         // Disable the limit
         .withConfiguration(config -> config.withMaxRequestBodySize(Map.of(ContentTypes.Form, -1)))
@@ -105,9 +105,9 @@ public void post_server_configuration_max_form_data(String scheme, boolean chunk
     // Large, enforce using default w/out a specific configuration for application/x-www-form-urlencoded
     withScheme(scheme)
         .withChunked(chunked)
-        .withBodyParameterCount(42 * 1024)  // 131,072, 131,072
+        .withBodyParameterCount(42 * 1024)
         .withBodyParameterSize(128)
-        // Disable the limit
+        // Remove the limit for form data, and fall back to the global
         .withConfiguration(config -> config.withMaxRequestBodySize(Map.of("*", 128 * 1024)))
         .expectResponse("""
             HTTP/1.1 413 \r