> `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

Reply via email to