Support multi-valued HTTP headers

[Former user]

HttpMessage needs to support multiple Set-Cookie headers as per the http specification.

Currently only one cookie is returned. (The last one found as headers are in a simple map).

https://developer.mozilla.org/en-US/docs/Web/HTTP/Cookies

[Vladimir Dzhuvinov]

Would you be interested in submitting a PR?

[Vladimir Dzhuvinov]

Breaking change, for 6.0 release

[Vladimir Dzhuvinov]

Implemented: c30600f8

To be released as v6.0 in couple of days

[Olaf Rode]

Hi this issue is not resolved yet. Currently only one value will be stored when processing the response received by HttpRequest.send() --> values.get(0). Currently this leads to problems as I'm loosing a 'Set-Cookie' value. :-(

        // Set headers
        for (Map.Entry<String,List<String>> responseHeader: conn.getHeaderFields().entrySet()) {

            if (responseHeader.getKey() == null) {
                continue; // skip header
            }

            List<String> values = responseHeader.getValue();
            if (values == null || values.isEmpty() || values.get(0) == null) {
                continue; // skip header
            }

            response.setHeader(responseHeader.getKey(), values.get(0));
        }

All the best

Olaf

[Connect2id OSS]

Apparently it wasn't completely implemented. Stay tuned!

[Connect2id OSS]

Had been fixed in e6f0c5f and released as v6.1.1

[Olaf Rode]

Thanks a lot.