-
Notifications
You must be signed in to change notification settings - Fork 906
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request]: Don't uplink decrypted precise location to MQTT #5404
Comments
We already do this on the official broker. I think the better approach here is to hard enforce the firmware not uplinking of MQTT packets without the OkToMQTT bit flag, under explicit consent. This will both give users the choice to opt-in and the freedom to continue to uplink precise data to private brokers as desired. Removing this functionality entirely would be a very controversial change. |
My understanding from chatting with Garth was that relying on "OK to MQTT" as consent for the app/node to transmit precise location on default keys and for any other user to relay that precise location to MQTT was insufficient, for two reasons:
|
We do not collect any PII. Any parties doing so are responsible for this data, not Meshtastic. Furthermore, implementing your change as described, given the open-source nature of the project, someone only needs to comment out a few lines of code restoring the uplink of this data, and based on the popularity of the feature, I fully expect this to occur. All of these efforts to try to prevent inadvertent uplink need a line drawn in the sand somewhere. My opinion is throwing the whole feature out is unacceptable. The firmware level enforcement of this relies entirely on good faith acting by uplink nodes.
While I agree about the uplink of this information to 3rd party brokers being a surprise (which we will close with this flag enforcement), you have to explicitly turn on precise position. The default configuration does not lead you down this path, so it shouldn't entirely be a surprise if you know that you're sending locations with full precision, and you know that the ecosystem of MQTT uplink exists. Perhaps the apps have more explicit warnings here about the information being broadcasted, but that is not the responsibility of the firmware. |
This is very much not what Garth seemed to believe in Discord yesterday, but I don't know who's right.
This wouldn't be the only line of defense; it would work alongside changes to the client apps that Garth already made in iOS and was seemingly planning for Android.
To be clear, this isn't throwing the whole feature out; it's specifically removing just enough data that the location isn't considered sensitive data.
To be fair, that flag enforcement code could be removed just as easily as my suggested location blurring code.
True, but...
...it's entirely possible for someone to not know about the ecosystem of MQTT uplink. Fundamentally, I think the ethical requirement here is to make it very clear to users when they're about to share their location publicly, that anyone at all can see it, whether it's precise or imprecise, whether they're granting permission for others to uplink it, and what that all means. Right now, a sufficiently motivated and technical user can figure this all out, but there's not a single, clear, meaningful choice for users who just want to get things working and/or aren't as technical. I'd be happy to try to write some copy explaining the impact of a user's settings clearly, whether or not precise location on default keys sticks around. |
Platform
Cross-Platform
Description
As part of tightening handling of precise location as PII, when uplinking a position packet to MQTT:
It's worth considering whether this behavior should differ when uplinking to private brokers; my instinct is that it should apply to private brokers as well, since part of the concern that inspired this bug was that data can be relayed between (some) brokers.
The text was updated successfully, but these errors were encountered: