Skip to content
This repository was archived by the owner on Nov 28, 2018. It is now read-only.

Commit 181374f

Browse files
committed
Merge remote-tracking branch 'origin/cleanup_header_parsing'
2 parents ad70dfc + 6652efd commit 181374f

File tree

2 files changed

+11
-15
lines changed

2 files changed

+11
-15
lines changed

lib/stomp.js

+10-14
Original file line numberDiff line numberDiff line change
@@ -37,25 +37,21 @@ function parse_command(data) {
3737
return command[0];
3838
};
3939

40-
function parse_headers(headers_str) {
41-
var these_headers = {},
42-
one_header = [],
43-
header_key = null,
44-
header_val = null,
45-
headers_split = headers_str.split('\n');
40+
function parse_headers(raw_headers) {
41+
var headers = {},
42+
headers_split = raw_headers.split('\n');
4643

4744
for (var i = 0; i < headers_split.length; i++) {
48-
one_header = headers_split[i].split(':');
49-
if (one_header.length > 1) {
50-
header_key = one_header.shift();
51-
header_val = one_header.join(':');
45+
var header = headers_split[i].split(':');
46+
if (header.length > 1) {
47+
var header_key = header.shift().trim();
48+
var header_val = header.join(':').trim();
5249
these_headers[header_key] = header_val;
50+
continue;
5351
}
54-
else {
55-
these_headers[one_header[0]] = one_header[1];
56-
}
52+
these_headers[header[0].trim()] = header[1].trim();
5753
}
58-
return these_headers;
54+
return headers;
5955
};
6056

6157
function parse_frame(chunk) {

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"name" : "stomp",
33
"description": "Implementation of the STOMP protocol in node.js",
44
"keywords": [ "STOMP", "messaging", "queue", "protocol" ],
5-
"version": "v0.0.9",
5+
"version": "v0.1.0",
66
"homepage": "https://github.com/benjaminws/stomp-js",
77
"author": "Benjamin W. Smith <benjaminws@just-another.net>",
88
"contributors" : [ "rofflwaffls@gmail.com" ],

0 commit comments

Comments
 (0)