Skip to content

Commit bb0d717

Browse files
authored
Merge pull request mkschreder#19 from addyh/fixes-issue-#16
Fixes issue mkschreder#16 (multiple Set-Cookie headers)
2 parents 78f812b + 1cf1ebb commit bb0d717

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

main.js

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ function runPHP(req, response, next, url, file) {
150150
if (/.*?\.php$/.test(file)) {
151151
var res = '', err = '';
152152

153-
var php = child.spawn(PHP_CGI, [], {
153+
var php = child.spawn(PHP_CGI.toString(), [], {
154154
env: env
155155
});
156156

@@ -190,6 +190,17 @@ function runPHP(req, response, next, url, file) {
190190
if (m[0] == 'Status') {
191191
response.statusCode = parseInt(m[1]);
192192
}
193+
if (m[0] == 'Set-Cookie') {
194+
var prevCookies = response.getHeader('Set-Cookie');
195+
if (prevCookies) {
196+
if (typeof prevCookies == 'string') {
197+
m[1] = [prevCookies, m[1]];
198+
} else {
199+
prevCookies.push(m[1]);
200+
m[1] = prevCookies;
201+
}
202+
}
203+
}
193204
if (m.length == 2) {
194205
response.setHeader(m[0], m[1]);
195206
}

0 commit comments

Comments
 (0)