feat: add MQTT configuration support via ADB intents#70
Open
harcerz wants to merge 2 commits intoRushB-fr:mainfrom
Open
feat: add MQTT configuration support via ADB intents#70harcerz wants to merge 2 commits intoRushB-fr:mainfrom
harcerz wants to merge 2 commits intoRushB-fr:mainfrom
Conversation
Adds 11 MQTT parameters (broker, port, username, password, client_id, base_topic, discovery_prefix, status_interval, allow_control, device_name, enabled) to ADB intent handling and JSON config mapping. MQTT password is stored securely in Android Keychain via the pending config bridge, same pattern as PIN.
Author
|
Sorry for the separate PR but I remembered about it when I had to prepare 7 tablets :) |
- Fix R8 obfuscation breaking Netty JCTools Unsafe field access by adding official HiveMQ ProGuard rules (-keepclassmembernames) - Fix silent error swallowing in autoStartMqtt() — errors now propagate to UI - Add connection error display and 15s timeout in MQTT settings - Add onConnectionError callback from native to JS for broker error messages - Fix password paste capturing only one character (slice(-1) → slice(-charsAdded)) - Fix broker URL field adding spaces after dots (keyboardType="url") - Add TLS auto-detection for port 8883 - Add password saved hint in MQTT settings - Catch Throwable instead of Exception in MqttModule.startMqtt()
17 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 Description
Adds 11 MQTT parameters (broker, port, username, password, client_id, base_topic, discovery_prefix, status_interval, allow_control, device_name, enabled) to ADB intent handling and JSON config mapping. MQTT password is stored securely in Android Keychain via the pending config bridge, same pattern as PIN.
This enables fully automated tablet provisioning with MQTT via ADB scripts, removing the need for manual UI configuration.
Usage
adb shell am start -n com.freekiosk/.MainActivity \ --es pin "1234" \ --es mqtt_enabled "true" \ --es mqtt_broker_url "broker.local" \ --es mqtt_port "1883" \ --es mqtt_username "user" \ --es mqtt_password "pass"🔧 Type of Change
✅ Testing
Device(s) tested:
📋 Checklist