search

benjaminws / stomp-js

Implementation of the STOMP protocol in node.js
BSD 3-Clause "New" or "Revised" License
87 stars 47 forks source link

stomp.js doesnt parse headers #7

Closed EricRossC closed 12 years ago

EricRossC commented 13 years ago

Hi there:

I think there's a typo in stomp.js, parse_headers(...):

var these_headers = [],

I think must be

var these_headers = {},

Using the original way, I never receive the headers (well, I receive an empty array). With this fix I started to receive the full headers.

Good library BTW!

benjaminws commented 13 years ago

Thanks for your find, I'm about to push up a branch with the fix.

benjaminws commented 13 years ago

Pushed the fix up to gix-headers branch. Can you try it out? Looks like you were right BTW. Good catch!

benjaminws commented 13 years ago

Err branch name is fix-headers.

benjaminws commented 12 years ago

Fixed this up in e4b316190ef9965466f459dcf99869b45d2cb787