search

spring-projects / spring-framework

Spring Framework
https://spring.io/projects/spring-framework
Apache License 2.0
56.72k stars 38.15k forks source link

Revisit `HttpHeaders` implementing the `MultiValueMap` API #33913

Open simonbasle opened 3 days ago

simonbasle commented 3 days ago

See #33823 : the MultiValueMap methods that iterate over the whole header collection misalign with some underlying server implementations.

On the other hand having HttpHeaders implement MultiValueMap brings in some (marginal?) benefits: using CollectionUtils, asserting headers with map-specific assertions, putting all the headers into a plain Map (map.putAll(headers)), etc...