Partially implement WifiEnumerateAccessPoints to expose access point ids #113
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Type
Side Effects
Goals
WifiEnumerateAccessPoints
API.Technical Details
Partially implement
WifiEnumerateAccessPoints
to surface discovered APs, populating their id (interface name) only.-v
flag fornetremote-server
.netremote-cli
to display discovered access points.LOG<E|W|I>
instead of those with full names likeLOG_<ERROR|WARNING|INFO>
.Test Results
netremote-cli
on a Linux machine with 2 AP-capable wi-fi devices with the following results:Reviewer Focus
Future Work
WifiEnumerateAccessPoints
results with access point capabilities. That portion is too large for this already sizable PR.AccessPointManager
to accept anIAccessPointFactory
to create access point instances instead of creating the base class typeAccessPoint
.Checklist
all
compiles cleanly.