search

friedrith / node-wifi

📶 NodeJS tool to manage wifi (connections, scans)
MIT License
395 stars 161 forks source link

Uncomplete list of AP over linux #64

Open waken22 opened 5 years ago

waken22 commented 5 years ago

Hi, thanks to read the issue, i didn't found a guideline to just open an issue. (Currently working on a project that uses node-wifi inside an IoT device, nmcli tool, version 1.2.6)

So I was having troubles to get an updated list of visible APs, not sure if my issue is related with network manager but I seems that i find a way to make it works using nmcli device wifi rescan before a wifi.scan(); only when i did that it works, so my question is if this is a specific workaround for me or it happens to someone else?

waken22 commented 5 years ago

I'm not sure if the rescan is needed but i found something that is giving me problems and is related by the title topic.

I want to ask what is the propose of this inside linux-scan.js and linux-current-connections.js

      if (config.iface) {
          lines.shift(); // ???? what
      }

I don't understand why is needed a shift here, it removes an AP from the array... and by that i'm not getting all the APs...

francoisrousseau commented 5 years ago

Amazing find waken22! I have the same issue... I also had to remove the shift inside "getCurrentConnection" from file linux-current-connections.js