RFR: 8268960: com/sun/net/httpserver/Headers.java: Ensure mutators normalize keys and disallow null for keys and values [v3]

Julia Boes jboes at openjdk.java.net
Wed Jun 23 18:02:02 UTC 2021


> `com.sun.net.httpserver.Headers` normalizes its keys to adhere to the following format: First character uppercase, all other characters lowercase, for example `"foo" -> "Foo"`. This behaviour is not consistent across the mutator methods of the class, in particular `putAll()` and `replaceAll()` do not apply normalization.
> 
> The suggested fix is to update the implementation of `putAll()` and to add an implementation of of the java.util.Map default method `replaceAll()`. While here, we can improve `equals()` by adding a type check and add a `toString()` implementation.
> 
> Additionally,  the Headers class is updated to disallow null values for keys and values.

Julia Boes has updated the pull request incrementally with one additional commit since the last revision:

  confirm HttpExchange::sendResponseHeaders fails if key/value null

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/4528/files
  - new: https://git.openjdk.java.net/jdk/pull/4528/files/6301bc1f..1e0ea047

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4528&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4528&range=01-02

  Stats: 49 lines in 2 files changed: 32 ins; 5 del; 12 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4528.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4528/head:pull/4528

PR: https://git.openjdk.java.net/jdk/pull/4528


More information about the net-dev mailing list