MMM-NetworkScanner-loading...indefinitely

sdetweil

@jthirasilpa said in MMM-NetworkScanner-loading...indefinitely:

[22:23:34.736] [ERROR] ReferenceError: options is not defined

but you added the 4 lines, right?

var options = {              
    cachePassword: true,
    prompt: 'Password, ???? '     // put your password where ???? are (notice the quotes around)
}

and then the updated line

 var arp = sudo(['arp-scan', '-q', arpHosts], options);

karsten13

what is the result of sudo arp-scan -l running on your machine?

sdetweil

@jthirasilpa just do this… get my version of the module

cd ~/MagicMirror/modules
mv MMM-NetworkScanner MMM-NetworkScanner.original
git clone https://github.com/sdetweil/MMM-NetworkScanner
cd MMM-NetworkScanner
npm install
cd ~/MagicMirror/config
nano config.js

add the following line to the config section of the network scanner module

Password: '??????',    // where '?????' is you password in quotes, for example 'raspberry', note the trailing comma

ctrl-o at save the config.js
ctrl-x to exit nano

Restart your MM

jthirasilpa

@sdetweil
Thanks Sam - tried your version of the module and modified the config.js accordingly. Still not quite working. Error logs as follows:

0|mm       |     at process._tickCallback (internal/process/next_tick.js:61:11)
0|mm       | [09:41:25.508] [ERROR]  ReferenceError: options is not defined
0|mm       |     at Class.scanNetworkMAC (/home/jojo/MagicMirror/modules/MMM-NetworkScanner/node_helper.js:42:54)
0|mm       |     at Class.socketNotificationReceived (/home/jojo/MagicMirror/modules/MMM-NetworkScanner/node_helper.js:29:18)
0|mm       |     at Socket.<anonymous> (/home/jojo/MagicMirror/js/node_helper.js:113:11)
0|mm       |     at Socket.emit (events.js:198:13)
0|mm       |     at /home/jojo/MagicMirror/node_modules/socket.io/lib/socket.js:528:12
0|mm       |     at process._tickCallback (internal/process/next_tick.js:61:11)
0|mm       | [09:41:55.508] [ERROR]  ReferenceError: options is not defined
0|mm       |     at Class.scanNetworkMAC (/home/jojo/MagicMirror/modules/MMM-NetworkScanner/node_helper.js:42:54)
0|mm       |     at Class.socketNotificationReceived (/home/jojo/MagicMirror/modules/MMM-NetworkScanner/node_helper.js:29:18)
0|mm       |     at Socket.<anonymous> (/home/jojo/MagicMirror/js/node_helper.js:113:11)
0|mm       |     at Socket.emit (events.js:198:13)
0|mm       |     at /home/jojo/MagicMirror/node_modules/socket.io/lib/socket.js:528:12
0|mm       |     at process._tickCallback (internal/process/next_tick.js:61:11)

sdetweil

@jthirasilpa hm… can u show me show me these lines from node_helper.js

scanNetworkMAC: function() {
        this.log(this.name + " is performing arp-scan");

        var self = this;
        // Target hosts/network supplied in config or entire localnet
        var arpHosts = this.config.network || '-l';
				var options = {              
						cachePassword: true,
						prompt: 'Password,' + this.config.Password     // put your password where ???? are (notice the quotes around)
				}
        var arp = sudo(['arp-scan', '-q', arpHosts], options);

I tested here with three different addresses… some coming online later… worked, no errors…

jthirasilpa

@karsten13
Hi - it seems to function correctly - it returns the IP and MAC addresses of the various clients on the network.

jthirasilpa

@sdetweil
I tried it both ways - but should the ‘password’ field in the helper file be filled in?

sdetweil

@jthirasilpa no… use the config.js

the reference to the options variable should be from line 47, not 42…

so, something is wrong with the code u have in node_helper.js

did u follow my steps above exactly?

jthirasilpa

Sorry I’m not being clear. I know that I need to modify the config.js (and have done so), but in line 45 of node_helper.js (as shown below) there is:

 prompt: 'Password,' + this.config.Password     // put your password where ???? are (notice the quotes around)

Should I leave this as is or fill in the password? That is where I have tried it both ways.

Code you asked for from node_helper.js:

    scanNetworkMAC: function() {
 38         this.log(this.name + " is performing arp-scan");
 39
 40         var self = this;
 41         // Target hosts/network supplied in config or entire localnet
 42         var arpHosts = this.config.network || '-l';
 43                                 var options = {
 44                                                 cachePassword: true,
 45                                                 prompt: 'Password,' + this.config.Password     // put your password where ???? are (notice the quotes around)
 46                                 }
 47         var arp = sudo(['arp-scan', '-q', arpHosts], options);

sdetweil

@jthirasilpa oh, i see I should have removed the comment… you do not need to change any code…

but the error does not match the code u are running now…

please do

pm2 stop 0
pm2 flush
pm2 start 0
pm2 logs

that error must have been from yesterday