Skip to content
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/Proposal: Provide ability to disable (through admin channel) pairing new BT devices #4949

Closed
lesykm opened this issue Sep 21, 2022 · 5 comments
Labels
enhancement New feature or request

Comments

@lesykm
Copy link
Contributor

lesykm commented Sep 21, 2022

Right now anyone who will find a node can bind their phone/other device to it and get access to the network.
It would be nice to add ability to completely turn off (and later optionally turn on) pairing of new devices to given nodes via admin channel commands.

Technically it will work through [Bluetooth enabled] parameter but that would work only for router-kind nodes. For end-user nodes usecase is - continue support of existing paired devices but don't allow new devices to join.

Another hacky way for screen-less devices is setting to random pin.

But again most convenient way would probably be - adding "bluetooth.mode" - "disabled". (while "bluetooth.enabled" - "true").
Also I suggest rename bluetooth.mode -> bluetooth.pairing_mode.

@caveman99 caveman99 added the enhancement New feature or request label Sep 21, 2022
@garthvh
Copy link
Member

garthvh commented Sep 21, 2022

You can disable Bluetooth already meshtastic --set bluetooth.enabled false, and set a fixed pin if you want to restrict access to only users that know that pin.

@lesykm
Copy link
Contributor Author

lesykm commented Sep 21, 2022

As I mentioned just disabling bluetooth will disable usage of the end nodes. Just disable pairing while enable bluetooth running will be better.
Regarding fixed pin - it will be visible on a screen, right?

@garthvh
Copy link
Member

garthvh commented Sep 23, 2022

The fixed pin works without a device screen fine, it defaults to 123456 and you can set it from the CLI, iOS or the webui. I am not aware of any way in the nimbleBLE stack on device or the various central managers used in client apps to have a device advertise but only to certain other devices.

@caveman99
Copy link
Member

The whitelist looks like a good approach on that. I can see the reasoning behind wanting to use bluetooth but disabling signing up new devices.
Bluetooth (or better BLE) in its current state was a major source of problems for a long time, that's why some things are like they are right now. It has improved a lot since we used newer versions of the BLE stacks on the device., so we might start enabling security features again.

@thebentern thebentern transferred this issue from meshtastic/firmware Dec 16, 2023
@fifieldt fifieldt transferred this issue from meshtastic/protobufs Oct 5, 2024
@garthvh garthvh closed this as not planned Won't fix, can't repro, duplicate, stale Oct 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants