Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update linux-connect.js #93

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Update linux-connect.js #93

wants to merge 1 commit into from

Conversation

FinlayDaG33k
Copy link

@FinlayDaG33k FinlayDaG33k commented Sep 19, 2019

Description

Start working on getting additional data as per #92.
Once finished, this PR should return some additional information when a connection has been established.

Motivation and Context

This change will remove the need of checking the current connections "manually" in order to find out certain things like the interface used and some details about the AP.

Usage examples

wifi.connect({options}, (net) => {
  // Do things with the received information
});

How Has This Been Tested?

Only on my local machine thus far

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

todo list:

  • Get interface name used
  • Get MAC for the interface used
  • Get IPv4 and IPv6 addresses used (along with things that come with it like the netmask)
  • Get SSID of the AP
  • Get bssid of the AP
  • Get MAC of the AP (equaling the bssid for retrocompatibility)
  • Get channel of the AP
  • Get frequency of the AP
  • Get signal level
  • Get signal quality in %
  • Get security used
  • Get security flags
  • Get current mode
  • Add compatibility with Mac and Windows

Start working on getting additional data
@valkheim
Copy link

Hi, do you plan to keep working on it ?

@FinlayDaG33k
Copy link
Author

@valkheim Under all the work I have found myself in related to other projects and finding a new dayjob, unfortunately not :\

I also think it might be better to scrap this specific PR in order to work on something like we've described here: #92 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants