VarSet: Nami should not chomp on all 'Variable:' keys

marcelog / Nami

Asterisk manager interface (ami) client for nodejs

http://marcelog.github.com/Nami

Apache License 2.0

99 stars 59 forks source link

VarSet: Nami should not chomp on all 'Variable:' keys #35

Closed xytis closed 9 years ago

xytis commented 9 years ago

Previously Nami chomped all lines containing 'variable' in the key and tried to fill variables hash with KEY=VALUE pairs from that. Unfortunately Asterisk has this one command, where

Variable: KEY
Value: VALUE

https://wiki.asterisk.org/wiki/display/AST/Asterisk+11+ManagerEvent_VarSet

To avoid this I added additional check to distinguish VarSet syntax from other commands and events.

marcelog commented 9 years ago

Thanks! :)

xytis commented 9 years ago

Can You package this change to npm?

marcelog commented 9 years ago

sure, done!

xytis commented 9 years ago

Thanks a lot :shipit: :)