TypeError: Cannot call method 'forEach' of undefined

Question

TypeError: Cannot call method 'forEach' of undefined

shabbir92 opened this issue 9 years ago · comments

[root@linx channels]# ./bin/channels
info - socket.io started

/opt/node-esl-master/examples/channels/node_modules/modesl/lib/esl/connection.js:594
doc.row.forEach(function(row) {
^
TypeError: Cannot call method 'forEach' of undefined
at /opt/node-esl-master/examples/channels/node_modules/modesl/lib/esl/connection.js:594:29
at Parser. (/opt/node-esl-master/examples/channels/node_modules/xml2js/lib/xml2js.js:255:20)
at Parser.emit (events.js:95:17)
at Object.saxParser.onclosetag (/opt/node-esl-master/examples/channels/node_modules/xml2js/lib/xml2js.js:225:24)
at emit (/opt/node-esl-master/examples/channels/node_modules/sax/lib/sax.js:615:33)
at emitNode (/opt/node-esl-master/examples/channels/node_modules/sax/lib/sax.js:620:3)
at closeTag (/opt/node-esl-master/examples/channels/node_modules/sax/lib/sax.js:861:5)
at Object.write (/opt/node-esl-master/examples/channels/node_modules/sax/lib/sax.js:1181:11)
at Parser.exports.Parser.Parser.parseString (/opt/node-esl-master/examples/channels/node_modules/xml2js/lib/xml2js.js:273:29)
at Parser.parseString (/opt/node-esl-master/examples/channels/node_modules/xml2js/lib/xml2js.js:6:61)

centos 7
[root@linx channels]# uname -a
Linux linx.lan 3.10.0-229.20.1.el7.x86_64 #1 SMP Tue Nov 3 19:10:07 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
kindly help me on this to resolve

Chad Engler · Answer 1 · Tue Dec 29 2015 07:04:43 GMT+0800 (China Standard Time)

Are you using the latest version? Line 594 isn't that line in the source.

shabbir92 · Answer 2 · Wed Dec 30 2015 13:14:55 GMT+0800 (China Standard Time)

i have installed with this way
Downloaded as zip from https://github.com/englercj/node-esl/
after unzip goto examples/channels npm install
when i started above error displayed on terminal
when i tried to replace files in examples/channels/node_modules/modesl/lib/esl/
with new ones in node-esl-master/lib/esl then i see diffrent names like connection.js Connection.js
when i modify the index file with name esl.js then it says Error: Cannot find module 'node-uuid'
i think npm is installing old version of modesl

Chad Engler · Answer 3 · Thu Dec 31 2015 00:39:12 GMT+0800 (China Standard Time)

Go to your project's directory and do: npm install modesl.

Then in your project require('modesl').

That should give you the latest version from the package registry, using the proper install method. npm takes care of getting all the dependencies for you, and you can even check that the version is correct. It should be v1.1.6, which after installing you can see with npm ls --depth 0

shabbir92 · Answer 4 · Thu Dec 31 2015 16:24:44 GMT+0800 (China Standard Time)

Thank you for help,
I am testing examples/channells and there https://github.com/englercj/node-esl/blob/master/examples/channels/package.json on line 12 "modesl": "0.x" this is why npm installing old version ,when i replaced this with "modesl": "1.x", now it is installed latest version,
npm ls --depth 0
channels@0.0.1 /opt/channels
├── express@3.21.2
├── eyes@0.1.8
├── modesl@1.1.6
├── socket.io@0.9.17
└── utile@0.3.0

and now error is

./bin/channels
info - socket.io started

events.js:72
throw er; // Unhandled 'error' event
^
TypeError: Cannot call method 'forEach' of undefined
at /opt/channels/node_modules/modesl/lib/esl/Connection.js:586:29
at Parser. (/opt/channels/node_modules/xml2js/lib/xml2js.js:483:18)
at Parser.emit (events.js:95:17)
at Object.onclosetag (/opt/channels/node_modules/xml2js/lib/xml2js.js:444:26)
at emit (/opt/channels/node_modules/sax/lib/sax.js:639:35)
at emitNode (/opt/channels/node_modules/sax/lib/sax.js:644:5)
at closeTag (/opt/channels/node_modules/sax/lib/sax.js:903:7)
at Object.write (/opt/channels/node_modules/sax/lib/sax.js:1293:13)
at Parser.exports.Parser.Parser.parseString (/opt/channels/node_modules/xml2js/lib/xml2js.js:502:31)
at Parser.parseString (/opt/channels/node_modules/xml2js/lib/xml2js.js:7:59)

Chad Engler · Answer 5 · Fri Jan 01 2016 02:00:02 GMT+0800 (China Standard Time)

Hmm, not sure. The only way this would happen (that I am aware of) is if the message from FSW was invalid. Can you let me see what the raw message looks like?

shabbir92 · Answer 6 · Fri Jan 01 2016 13:30:25 GMT+0800 (China Standard Time)

Found problem on https://github.com/englercj/node-esl/blob/master/lib/esl/Connection.js#L578
parsed.rowCount = parseInt(doc.$.rowCount, 10);
and replaced with
parsed.rowCount = parseInt(doc.$.row_count, 10);
and ON
https://github.com/englercj/node-esl/blob/master/lib/esl/Connection.js#L585
} else {
replaced with
} else if (parsed.rowCount > 1){
And Now it working Thank You For help

Chad Engler · Answer 7 · Sat Jan 02 2016 01:02:04 GMT+0800 (China Standard Time)

What version of FSW are you using? Did they change the XML format recently?

shabbir92 · Answer 8 · Sat Jan 02 2016 14:16:21 GMT+0800 (China Standard Time)

"What version of FSW are you using?"
FreeSWITCH (Version 1.7.0 git 3b538d0 2015-12-16 08:07:19Z 64bit)

"Did they change the XML format recently?"
i have executed in fs_cli ' show channels as xml'
and result is below
< result row_count="0"/>
any other info you need ?

Chad Engler · Answer 9 · Sun Jan 03 2016 00:45:09 GMT+0800 (China Standard Time)

can you do "npm ls --depth 0" and paste the output so I can see the version of dependencies you are using?

I think xml2js made a breaking change in a recent version. Try changing the version from "xml2js": "~0.4", to "xml2js": "0.4.6", in the package.json and reinstall the dependency. After confirming it is the proper version with npm ls --depth 0 try again and see if you get this error (using the original unmodified source, of course).

shabbir92 · Answer 10 · Sun Jan 03 2016 17:06:48 GMT+0800 (China Standard Time)

i have tried as you asked ; with unmodified source
npm ls | grep xml2js
└─┬ xml2js@0.4.6
[root@linx channels]# npm ls --depth 0
channels@0.0.1 /opt/channels
├── express@3.21.2
├── eyes@0.1.8
├── modesl@1.1.6
├── socket.io@0.9.17
└── utile@0.3.0
[root@linx channels]# ./bin/channels
info - socket.io started

events.js:72
throw er; // Unhandled 'error' event
^
TypeError: Cannot call method 'forEach' of undefined
at /opt/channels/node_modules/modesl/lib/esl/Connection.js:586:29
at Parser. (/opt/channels/node_modules/modesl/node_modules/xml2js/lib/xml2js.js:428:18)
at Parser.emit (events.js:95:17)
at Object.onclosetag (/opt/channels/node_modules/modesl/node_modules/xml2js/lib/xml2js.js:397:26)
at emit (/opt/channels/node_modules/modesl/node_modules/sax/lib/sax.js:615:33)
at emitNode (/opt/channels/node_modules/modesl/node_modules/sax/lib/sax.js:620:3)
at closeTag (/opt/channels/node_modules/modesl/node_modules/sax/lib/sax.js:861:5)
at Object.write (/opt/channels/node_modules/modesl/node_modules/sax/lib/sax.js:1182:11)
at Parser.exports.Parser.Parser.parseString (/opt/channels/node_modules/modesl/node_modules/xml2js/lib/xml2js.js:447:31)
at Parser.parseString (/opt/channels/node_modules/modesl/node_modules/xml2js/lib/xml2js.js:6:59)

when i installed xml2js 0.4.6 or 0.4.15 error is same
i think error is because FSW is sending this < result row_count="0"/ > and
You are looking for rowCount instead of row_count and this is problem when parseInt function looks for expected string who is not there it returns NaN when i printed this variable via console.log
then printed value was NaN

Chad Engler · Answer 11 · Mon Jan 04 2016 02:07:44 GMT+0800 (China Standard Time)

That is definitely the error, what I am trying to figure out is what has changed. That line of code has been working for years, so something is different in your configuration.

I'm trying determine if we need to change it to row_count in the source, but more importantly why it needs to be changed so that it can still work for all the people it has apparently been working for.

shabbir92 · Answer 12 · Mon Jan 04 2016 15:26:42 GMT+0800 (China Standard Time)

Today i have aded ' console.log("doc:",doc)' before https://github.com/englercj/node-esl/blob/master/lib/esl/Connection.js#L578 To Look raw message
doc: { '$': { row_count: '0' } }
debug - websocket writing 5:::{"name":"data","args":[{"uuid":null,"data":{"rows":{}}}]}
doc: { '$': { row_count: '1' },
row:
{ '$': { row_id: '1' },
uuid: '6ba91232-b2b2-11e5-9760-6d9fe5d3764d',
direction: 'inbound',
created: '2016-01-04 12:11:48',
created_epoch: '1451891508',
name: 'skypopen/skype103',
state: 'CS_EXECUTE',
cid_name: '',
cid_num: '',
ip_addr: '',
dest: 'skype_in',
application: 'sleep',
application_data: '2000',
dialplan: 'XML',
context: 'default',
read_codec: 'L16',
read_rate: '16000',
read_bit_rate: '256000',
write_codec: 'L16',
write_rate: '16000',
write_bit_rate: '256000',
secure: '',
hostname: 'linx.lan',
presence_id: '',
presence_data: '',
callstate: 'ACTIVE',
callee_name: '',
callee_num: '',
callee_direction: '',
call_uuid: '',
sent_callee_name: '',
sent_callee_num: '',
initial_cid_name: '',
initial_cid_num: '',
initial_ip_addr: '',
initial_dest: 'skype_in',
initial_dialplan: 'XML',
initial_context: 'default' } }
debug - websocket writing 5:::{"name":"data","args":[{"uuid":null,"data":{"rows":{"6ba91232-b2b2-11e5-9760-6d9fe5d3764d":{"uuid":"6ba91232-b2b2-11e5-9760-6d9fe5d3764d","direction":"inbound","created":"2016-01-04 12:11:48","created_epoch":"1451891508","name":"skypopen/skype103","state":"CS_EXECUTE","cid_name":"","cid_num":"","ip_addr":"","dest":"skype_in","application":"sleep","application_data":"2000","dialplan":"XML","context":"default","read_codec":"L16","read_rate":"16000","read_bit_rate":"256000","write_codec":"L16","write_rate":"16000","write_bit_rate":"256000","secure":"","hostname":"linx.lan","presence_id":"","presence_data":"","callstate":"ACTIVE","callee_name":"","callee_num":"","callee_direction":"","call_uuid":"","sent_callee_name":"","sent_callee_num":"","initial_cid_name":"","initial_cid_num":"","initial_ip_addr":"","initial_dest":"skype_in","initial_dialplan":"XML","initial_context":"default"}}}}]}

And tried to look into Sources At https://freeswitch.org/stash/projects/FS/repos/freeswitch/browse/src/mod/applications/mod_commands/mod_commands.c
Unfortunatly Not found Anywhere in v1.2 or v1.4 or v1.6 or v1.7 rowCount Only Found row_count
And i am wonder how this was Working before ?

shabbir92 · Answer 13 · Mon Jan 04 2016 15:55:00 GMT+0800 (China Standard Time)

To figure Out why this is Happend
I have tried To look into sources And Downloaded modesl v0.0.11 at line 586 Found
parsed.row_count = parseInt(doc['$'].row_count, 10);
But in v 1.1.1 https://github.com/englercj/node-esl/blob/v1.1.1/lib/esl/Connection.js#L581
it is changed to
parsed.rowCount = parseInt(doc.$.rowCount, 10);
and this is problem (what i understand)

Chad Engler · Answer 14 · Tue Jan 05 2016 00:52:10 GMT+0800 (China Standard Time)

I'm dumb, I should've just looked at the blame in the first place. So sorry about all the trouble, a new version is published that should fix your issue.

shabbir92 · Answer 15 · Tue Jan 05 2016 18:13:53 GMT+0800 (China Standard Time)

Dear brother hello,
You have forgot to replace https://github.com/englercj/node-esl/blob/master/lib/esl/Connection.js#L585
with
} else if (parsed.rowCount > 1){
Due to this problem not solved because sometimes row count is '0'
[root@linx channels]# npm ls --depth 0
channels@0.0.1 /opt/channels
├── express@3.21.2
├── eyes@0.1.8
├── modesl@1.1.7
├── socket.io@0.9.17
└── utile@0.3.0

[root@linx channels]# ./bin/channels
info - socket.io started

events.js:72
throw er; // Unhandled 'error' event
^
TypeError: Cannot call method 'forEach' of undefined
at /opt/channels/node_modules/modesl/lib/esl/Connection.js:586:29
at Parser. (/opt/channels/node_modules/xml2js/lib/xml2js.js:483:18)
at Parser.emit (events.js:95:17)
at Object.onclosetag (/opt/channels/node_modules/xml2js/lib/xml2js.js:444:26)
at emit (/opt/channels/node_modules/sax/lib/sax.js:639:35)
at emitNode (/opt/channels/node_modules/sax/lib/sax.js:644:5)
at closeTag (/opt/channels/node_modules/sax/lib/sax.js:903:7)
at Object.write (/opt/channels/node_modules/sax/lib/sax.js:1293:13)
at Parser.exports.Parser.Parser.parseString (/opt/channels/node_modules/xml2js/lib/xml2js.js:502:31)
at Parser.parseString (/opt/channels/node_modules/xml2js/lib/xml2js.js:7:59)

kindly update it again and please if u Don't mind also update
https://github.com/englercj/node-esl/blob/master/examples/channels/package.json#L12
Thank You

Chad Engler · Answer 16 · Wed Jan 06 2016 00:47:46 GMT+0800 (China Standard Time)

You're right, fixed now hopefully!