On 2019/11/07 11:10, Lauri Tirkkonen wrote: > I think it might be a little confusing to make this operation implicit > in another operation (ie. list APs), and for root only. How does a user > find out about this feature?
The manpage part of the diff is yet to be written ;) > Wouldn't it be better to have a separate > command to trigger the scan, eg. 'rescan'? That could be documented and > restricted to root. But the command is not "ifconfig list_aps", it's "ifconfig scan" and it's really expected that this would trigger a scan. To me the current behaviour (where "ifconfig iwm0 scan" might give you a list from 300km while you were travelling connected to your phone hotspot) with no way to update it short of forcing a disconnection is more confusing. It's not ideal that "ifconfig scan" prints an old list at all, but that is the current status and this diff doesn't change that. Still it's an improvement. Could it be made better still? Yes, but I don't see that as a reason to hold off on making *this* change.
