Skip to content

A couple of fixes for BeaconDB

Chris T requested to merge kop316/geoclue:wip/kop316/beacondb into master

Hello!

This MR has a couple of fixes for compatibility with BeaconDB:

  1. For both the geolocate and submit, the query includes the SSID so that BeaconDB can process it (this is also in line with the ichnaea https://ichnaea.readthedocs.io/en/latest/api/geolocate.html and https://ichnaea.readthedocs.io/en/latest/api/geosubmit2.html )
  2. This replaces rather than appends the User-Agent Heading. For some reason you have to replace it, not append it to change the user agent (I had to do the same thing with mmsd-tng with replace: https://gitlab.com/kop316/mmsd/-/blob/master/src/service.c?ref_type=heads#L3884 )

I tested this successfully with BeaconDB and confirmed it worked with @joelkoen .

Edited by Chris T

Merge request reports

Loading