Bug 22378

Summary:	Better AP MAC detection
Product:	GeoClue	Reporter:	Bastien Nocera <bugzilla>
Component:	General	Assignee:	Geoclue Bugs <geoclue-bugs>
Status:	RESOLVED FIXED	QA Contact:	Geoclue Bugs <geoclue-bugs>
Severity:	normal
Priority:	medium	CC:	pbrobinson, ross
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:
Attachments:	0001-Use-NetworkManager-to-export-the-AP-MAC-address.patch

Description Bastien Nocera 2009-06-19 16:29:51 UTC

1. Add a RequiresWifiNetwork type to the provider
2. Add API to the connectivity object to get the BSSID [1]
3. Pass the BSSID to the backends for them to use (plazes, localnet and skyhook)

This would allow better error reporting (as in, we know it wouldn't work on wired LANs).

For 3., we might also be able to pass lists of known BSSIDs, which might help with detection (or at least allow detecting location using WLAN data even though we're connected via a WWAN).

Comments?

[1]: This is easy with NetworkManager, as seen in nm-tool.

Comment 1 Bastien Nocera 2009-06-20 15:12:24 UTC

s/BSSID/AP MAC/

Comment 2 Bastien Nocera 2009-06-23 06:40:13 UTC

Created attachment 27050 [details] [review]
0001-Use-NetworkManager-to-export-the-AP-MAC-address.patch

Comment 3 Ross Burton 2009-06-24 07:07:25 UTC

Bastien: as discussed on chat change the BSSID logic and I think it's good for committing.

Comment 4 Keith Preston 2009-08-17 09:03:11 UTC

Looks good to me, I have pushed it into the master branch, Thanks for the patch

Comment 5 Bastien Nocera 2009-08-20 03:52:24 UTC

Forgot one detail, the code needs a slight change to use the currently associated AP MAC instead of just the strongest one (or the strongest of the associated AP signal, if there's multiple ones).

Comment 6 Pierre-Luc Beaudoin 2009-08-20 11:49:09 UTC

(In reply to comment #5)
> Forgot one detail, the code needs a slight change to use the currently
> associated AP MAC instead of just the strongest one.

I think using the strongest one is a reasonable choice.  You might be get better location results on that.

Comment 7 Bastien Nocera 2010-01-29 05:18:20 UTC

Fine then. Closing as the code already does that.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.