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

2021-06-25 Thread Julia Boes
On Thu, 24 Jun 2021 15:15:07 GMT, Daniel Fuchs wrote: >> Julia Boes has updated the pull request incrementally with one additional >> commit since the last revision: >> >> confirm HttpExchange::sendResponseHeaders fails if key/value null > >

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

2021-06-24 Thread Michael McMahon
On Wed, 23 Jun 2021 18:02:02 GMT, Julia Boes wrote: >> `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

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

2021-06-24 Thread Daniel Fuchs
On Wed, 23 Jun 2021 18:02:02 GMT, Julia Boes wrote: >> `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

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

2021-06-24 Thread Chris Hegarty
On Wed, 23 Jun 2021 18:02:02 GMT, Julia Boes wrote: >> `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

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

2021-06-23 Thread Julia Boes
> `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