Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple HTTP header values #4

Closed
eserte opened this issue Sep 9, 2009 · 3 comments
Closed

Multiple HTTP header values #4

eserte opened this issue Sep 9, 2009 · 3 comments

Comments

@eserte
Copy link

eserte commented Sep 9, 2009

I think the spec should say something about multiple values for HTTP values in the environment. As the spec states that values in the environment must be scalar strings, such values should probably be concatenated using commas, like suggested in RFC 2616.

@miyagawa
Copy link
Member

miyagawa commented Sep 9, 2009

Yes, it's covered in the FAQ:

Note that I'm talking about multiple header lines with the same key,
and NOT about multiple header values (e.g. C<Accept: text/html,
text/plain, *>). Joining the header values with C<, > is obviously
the applications' job. LHTTP::Headers does exactly that when it's
passed an array reference as a header value, for instance.

@miyagawa
Copy link
Member

This is a very good point and indeed there were bugs in a couple of implementations, such as AnyEvent and ServerSimple:
http://github.com/miyagawa/http-server-simple/commit/b505e06e2f00adf88feefecbc0cfa563aaa6713c

@miyagawa
Copy link
Member

feedback merged.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants