Skip to content

Support multiple Content-Length values #2470

Closed
@ghostd

Description

@ghostd

Hi,

I try to work on some failing tests of servo. To summarize the issue : the fetch specification has been recently updated to handle multiple values for the Content-Length header. I found Hyper client does not handle the header as expected by the specification. To be more precise, hyper does not handle the multiple values into only one header entry : Content-Length: 42,42. I don't know if we want to update hyper to follow this specification or not.

I reproduced the error with the example client and the following node http server code:

var http = require('http');

http.createServer(function (req, res) {
    res.writeHead(200, {'Content-Length': '42,42'});
    res.write('012345678901234567890123456789012345678901');
    res.end();
}).listen(8080);

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-http1Area: HTTP/1 specific.C-bugCategory: bug. Something is wrong. This is bad!E-easyEffort: easy. A task that would be a great starting point for a new contributor.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions