search

qiwi / qorsproxy

Yet another cors proxy
MIT License
5 stars 3 forks source link

Support multiple same-name headers #14

Closed Khromushkin closed 5 years ago

Khromushkin commented 5 years ago

The most common case is Set-Cookie headers

Khromushkin commented 5 years ago

I see that it was designed to support array as header value and there exists a test for that functionality. https://github.com/qiwi/qorsproxy/blob/master/test/servlet/corsproxy/middleware/mutator.js#L75

But now we have only first Set-Cookie header passing through proxy. So this issue is actually not a feature request, but a bug report.

Khromushkin commented 5 years ago

This issue relates to the request module. There are some issues with no activity for a long time. For example https://github.com/request/request/issues/3086

Khromushkin commented 5 years ago

fixed in #20