Skip to content
This repository has been archived by the owner on Apr 8, 2019. It is now read-only.

getCurrentSSID() returns SSID even when not connected (getCurrentBSSID does not exist) #7

Closed
tripflex opened this issue Jan 5, 2018 · 4 comments
Assignees
Labels

Comments

@tripflex
Copy link

tripflex commented Jan 5, 2018

Seems that getCurrentSSID returns an SSID even when it's not connected to that network, which calls the getConnectedSSID

see: https://stackoverflow.com/questions/41366481/android-why-wifimanager-getconnectioninfo-getssid-return-disconnected-ssid
https://developer.android.com/reference/android/net/wifi/WifiInfo.html#getSSID()

Trying to use this to handle the ssidToDisconnect

Reading the docs it seems that getCurrentBSSID would work (as it should return null if not connected), but even though that method is available, it does not exist in main java file (which I found is actually related to the WiFiWizard, not your edits)

@arsenal942 arsenal942 added the bug label Jan 6, 2018
@arsenal942 arsenal942 self-assigned this Jan 6, 2018
@arsenal942
Copy link
Owner

Good spot. This goes back to the OG WifiWizard code being a bit trash.

I'm just going to re-write the entire Android platform side since it makes a lot more sense to do so.

@arsenal942
Copy link
Owner

Working on this now.

arsenal942 pushed a commit that referenced this issue Mar 20, 2018
#7
- Check has been implemented which will look if the ssid is unknown and throw an error (false) if it is. The android doco literally specifies this is a string of '<unknown ssid>'....how dumb is that...
arsenal942 pushed a commit that referenced this issue Mar 20, 2018
#7
- Update package and readme to 2.3.2
arsenal942 pushed a commit that referenced this issue Mar 20, 2018
arsenal942 pushed a commit that referenced this issue Mar 20, 2018
…n earth would we assign it a blank MAC address....

- Implemented the function 'getConnectedBSSID' for Android since it was missing
@arsenal942
Copy link
Owner

arsenal942 commented Mar 20, 2018

@tripflex 'getConnectedBSSID' now exists and the OG getConnectedSSID has been modified. Check #3deb50f to see the changes.

@arsenal942
Copy link
Owner

Release v2.4.0 fixed.

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

No branches or pull requests

2 participants