Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Support multiple 'link' headers #1187

Closed
mfncooper opened this issue Jun 15, 2011 · 3 comments
Closed

Support multiple 'link' headers #1187

mfncooper opened this issue Jun 15, 2011 · 3 comments

Comments

@mfncooper
Copy link

The HTTP 'link' header may legally appear multiple times in a request, but Node will drop all but the first one.

The trivial fix is to add 'link' to the case statement here:

https://github.com/joyent/node/blob/master/lib/http.js#L287

@isaacs
Copy link

isaacs commented Jun 15, 2011

+1. You'll almost always have more than one, in fact. Wanna sign the cla (if you haven't) and try a patch? (If you don't feel like it, I or someone else can, I'm sure.)

@mfncooper
Copy link
Author

Sure. My CLA should be on file already. I posted the patch here:

http://groups.google.com/group/nodejs-dev/msg/2afa0dfc02831a36

@ry ry closed this as completed in d627083 Jun 20, 2011
@ry
Copy link

ry commented Jun 20, 2011

thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants