Fixed cors headers.

petersirka · petersirka · commit a5beb5b3f2a9 · 2016-01-26T15:33:59.000+01:00
diff --git a/index.js b/index.js
@@ -1137,7 +1137,7 @@ Framework.prototype.cors = function(url, flags, credentials) {
 					methods.push(flag.toUpperCase());
 					break;
 				default:
-					headers.push(flag);
+					headers.push(flags[i]);
 					break;
 			}
 		}
diff --git a/test/controllers/default.js b/test/controllers/default.js
@@ -145,7 +145,7 @@ exports.install = function() {
     framework.cors('/api/*');
     framework.cors('/cors/origin-all/');
     framework.cors('/cors/origin-not/', ['http://www.petersirka.eu', 'http://www.858project.com']);
-    framework.cors('/cors/headers/', ['post', 'put', 'delete', 'options'], true);
+    framework.cors('/cors/headers/', ['post', 'put', 'delete', 'options', 'X-Ping'], true);
 };
 
 function plain_options() {
diff --git a/test/test-framework-debug.js b/test/test-framework-debug.js
@@ -151,6 +151,7 @@ function test_routing(next) {
 			assert.ok(headers['access-control-allow-origin'] === 'http://www.petersirka.eu', 'CORS, headers problem 1');
 			assert.ok(headers['access-control-allow-credentials'] === 'true', 'CORS, headers problem 2');
 			assert.ok(headers['access-control-allow-methods'] === 'POST, PUT, DELETE, OPTIONS', 'CORS, headers problem 3');
+			assert.ok(headers['access-control-allow-headers'] === 'X-Ping', 'CORS, headers problem 4');
 			complete();
 		}, null, { 'origin': 'http://www.petersirka.eu' });
 	});
diff --git a/test/test-framework-release.js b/test/test-framework-release.js
@@ -151,6 +151,7 @@ function test_routing(next) {
 			assert.ok(headers['access-control-allow-origin'] === 'http://www.petersirka.eu', 'CORS, headers problem 1');
 			assert.ok(headers['access-control-allow-credentials'] === 'true', 'CORS, headers problem 2');
 			assert.ok(headers['access-control-allow-methods'] === 'POST, PUT, DELETE, OPTIONS', 'CORS, headers problem 3');
+			assert.ok(headers['access-control-allow-headers'] === 'X-Ping', 'CORS, headers problem 4');
 			complete();
 		}, null, { 'origin': 'http://www.petersirka.eu' });
 	});

Original file line number	Diff line number	Diff line change
`@@ -1137,7 +1137,7 @@ Framework.prototype.cors = function(url, flags, credentials) {`
`1137`	`1137`	`methods.push(flag.toUpperCase());`
`1138`	`1138`	`break;`
`1139`	`1139`	`default:`
`1140`		`- headers.push(flag);`
	`1140`	`+ headers.push(flags[i]);`
`1141`	`1141`	`break;`
`1142`	`1142`	`}`
`1143`	`1143`	`}`