MQTT client is working perfectly now! MQTT on devices are working now.

MQTT.md

@@ -0,0 +1,46 @@
+###### default.conf
+```
+# Insecure
+listener 1883
+
+# TLS/SSL
+listener 8883
+acl_file /etc/mosquitto/conf.d/aclfile
+protocol mqtt
+require_certificate false
+certfile /etc/mosquitto/certs/cert.pem
+cafile /etc/mosquitto/certs/fullchain.pem
+keyfile /etc/mosquitto/certs/privkey.pem
+
+listener 8083
+protocol websockets
+certfile /etc/mosquitto/certs/cert.pem
+cafile /etc/mosquitto/certs/fullchain.pem
+keyfile /etc/mosquitto/certs/privkey.pem
+```
+
+###### /etc/mosquitto/conf.d/aclfile
+```
+user acidvegas
+topic readwrite msh/#
+
+user mate
+topic readwrite msh/#
+
+pattern write $SYS/broker/connection/%c/state
+```
+
+###### mosquito.conf
+```
+pid_file /run/mosquitto/mosquitto.pid
+
+per_listener_settings true
+allow_anonymous false
+persistence true
+persistence_location /var/lib/mosquitto
+password_file /etc/mosquitto/passwd
+
+log_dest file /var/log/mosquitto/mosquitto.log
+
+include_dir /etc/mosquitto/conf.d
+```

README.md

@@ -14,7 +14,7 @@ The hardware I am experimenting with: [Lilygo T-Deck](https://www.lilygo.cc/prod
 
 ## Bugs & Issues
 - Devices must have Wifi turned off when going mobile. Upon leaving my house with WiFi still enabled, the UI & connection was EXTREMELY laggy & poor. Couldn't even type well...
-- Devices using a custom MQTT with TLS & auth will cause a reboot loop *(Need to fix this ASAP)*
+- Devices using a MQTT with TLS will reboot loop.
 - A fix for the reboot loop is simply disabling MQTT over serial with `meshtastic --set mqtt.enabled false`
 - `event_node` event is called **AS** we are defining the interface, so using `self.interface` in that callback will error.
 

SETUP.md

@@ -46,7 +46,7 @@ meshtastic --set network.wifi_enabled true --set network.wifi_ssid "CHANGEME" --
 
 ###### MQTT
 ```
-meshtastic --set mqtt.enabled true --set mqtt.address "CHANGEME" --set mqtt.username "CHANGEME" --set mqtt.password "CHANGEME" --set mqtt.tls_enabled true --set mqtt.root "msh/CHANGEME" --set mqtt.encryption_enabled true
+meshtastic --set mqtt.enabled true --set mqtt.address changeme --set mqtt.username changeme --set mqtt.password changeme --set mqtt.encryption_enabled true --set mqtt.tls_enabled false --set mqtt.root msh --set mqtt.map_reporting_enabled true --set mqtt.json_enabled true
 ```
 
 ###### BLUETOOTH

meshirc.py

@@ -7,7 +7,7 @@ import logging
 import ssl
 import time
 
-# EF576MkXA3aEURbCfNn6p0FfZdua4I
+# 0xEF576MkXA3aEURbCfNn6p0FfZdua4I
 
 # Formatting Control Characters / Color Codes
 bold        = '\x02'

meshmqtt.py

@@ -1,4 +1,4 @@
-#!/usr/bin/env python3
+#!/usr/bin/env python
 # Meshtastic MQTT Interface - Developed by acidvegas in Python (https://acid.vegas/meshtastic)
 
 import argparse
@@ -37,7 +37,9 @@ def decode_encrypted(message_packet):
 	'''
 	Decrypt an encrypted message packet.
 
-	:param message_packet: The message packet to decrypt'''
+	:param message_packet: The message packet to decrypt
+ 	'''
+
 	try:
 		key_bytes = base64.b64decode(key.encode('ascii'))
 
@@ -79,6 +81,29 @@ def decode_encrypted(message_packet):
 		logging.error(f'Failed to decrypt message: {str(e)}')
 
 
+def encrypt_message(channel, key, mesh_packet, encoded_message):
+    '''
+    Encrypt a message packet.
+    
+    :param channel: The channel to encrypt the message for
+    :param key: The encryption key
+    :param mesh_packet: The mesh packet to encrypt
+    :param encoded_message: The encoded message to encrypt
+    '''
+    
+    mesh_packet.channel = generate_hash(channel, key)
+    key_bytes = base64.b64decode(key.encode('ascii'))
+    nonce_packet_id = mesh_packet.id.to_bytes(8, "little")
+    nonce_from_node = node_number.to_bytes(8, "little")
+    nonce = nonce_packet_id + nonce_from_node
+
+    cipher = Cipher(algorithms.AES(key_bytes), modes.CTR(nonce), backend=default_backend())
+    encryptor = cipher.encryptor()
+    encrypted_bytes = encryptor.update(encoded_message.SerializeToString()) + encryptor.finalize()
+
+    return encrypted_bytes
+
+
 def on_connect(client, userdata, flags, rc, properties):
 	'''
 	Callback for when the client receives a CONNACK response from the server.
@@ -105,27 +130,68 @@ def on_message(client, userdata, msg):
 	:param msg: An instance of MQTTMessage. This is a
 	'''
 
+	print(f'{msg.topic}: {msg.payload}')
+
 	service_envelope = mqtt_pb2.ServiceEnvelope()
 
 	try:
 		service_envelope.ParseFromString(msg.payload)
-		# print(service_envelope)
+		print(service_envelope)
 		message_packet = service_envelope.packet
-		# print(message_packet)
+		print(message_packet)
 	except Exception as e:
-		logging.error(f'Failed to parse message: {str(e)}')
+		#logging.error(f'Failed to parse message: {str(e)}')
 		return
 
 	if message_packet.HasField('encrypted') and not message_packet.HasField('decoded'):
 		decode_encrypted(message_packet)
 
 
+def on_subscribe(client, userdata, mid, reason_code_list, properties):
+	'''
+	Callback for when the client receives a SUBACK response from the server.
+
+	:param client: The client instance for this callback
+	:param userdata: The private user data as set in Client() or user_data_set()
+	:param mid: The message ID of the subscribe request
+	:param reason_code_list: A list of SUBACK reason codes
+	:param properties: The properties returned by the broker
+	'''
+
+	# Since we subscribed only for a single channel, reason_code_list contains
+	# a single entry
+	if reason_code_list[0].is_failure:
+		print(f"Broker rejected you subscription: {reason_code_list[0]}")
+	else:
+		print(f"Broker granted the following QoS: {reason_code_list[0].value}")
+
+
+def on_unsubscribe(client, userdata, mid, reason_code_list, properties):
+	'''
+	Callback for when the client receives a UNSUBACK response from the server.
+
+	:param client: The client instance for this callback
+	:param userdata: The private user data as set in Client() or user_data_set()
+	:param mid: The message ID of the unsubscribe request
+	:param reason_code_list: A list of UNSUBACK reason codes
+	:param properties: The properties returned by the broker
+	'''
+
+	# Be careful, the reason_code_list is only present in MQTTv5.
+	# In MQTTv3 it will always be empty
+	if len(reason_code_list) == 0 or not reason_code_list[0].is_failure:
+		print("unsubscribe succeeded (if SUBACK is received in MQTTv3 it success)")
+	else:
+		print(f"Broker replied with failure: {reason_code_list[0]}")
+	client.disconnect()
+
+
 
 if __name__ == '__main__':
 	parser = argparse.ArgumentParser(description='Mesh MQTT')
 	parser.add_argument('--broker', default='mqtt.meshtastic.org', help='MQTT broker address')
 	parser.add_argument('--port', default=1883, type=int, help='MQTT broker port')
-	parser.add_argument('--root', default='msh/US/2/c', help='Root topic')
+	parser.add_argument('--root', default='#', help='Root topic')
 	parser.add_argument('--tls', action='store_true', help='Enable TLS/SSL')
 	parser.add_argument('--username', default='meshdev', help='MQTT username')
 	parser.add_argument('--password', default='large4cats', help='MQTT password')
@@ -137,15 +203,31 @@ if __name__ == '__main__':
 	replaced_key = padded_key.replace('-', '+').replace('_', '/')
 	key          = replaced_key
 
-	broadcast_id = 4294967295
+	broadcast_id = 4294967295 # Do we need to change this for a custom channel?
 
-	# client = mqtt.Client(client_id='', clean_session=True, userdata=None)
+	# Create the MQTT client
-	client = mqtt.Client(mqtt.CallbackAPIVersion.VERSION2)
+	client = mqtt.Client(mqtt.CallbackAPIVersion.VERSION2, client_id='', clean_session=True, userdata=None) # Defaults to mqtt.MQTTv311 (change with protocol=mqtt.MQTTv5)
-	client.on_connect = on_connect
+
+	# Set the authentication details
 	client.username_pw_set(username=args.username, password=args.password)
-	client.connect(args.broker, args.port, 60)
+
+	# Enable TLS/SSL if the --tls flag is set
+	if args.tls:
+		import ssl
+		client.tls_set(cert_reqs=ssl.CERT_REQUIRED, tls_version=ssl.PROTOCOL_TLSv1_2)
+		client.tls_insecure_set(False)
+
+	# Set the callbacks
+	client.on_connect     = on_connect
 	client.on_message     = on_message
+	client.on_subscribe   = on_subscribe
+	client.on_unsubscribe = on_unsubscribe
+
+	# Connect to the broker
+	client.connect(args.broker, args.port, 60)
+
+	# Subscribe to the root topic
 	client.subscribe(args.root, 0)
 
-	while client.loop() == 0:
+	# Start the keep-alive loop
-		pass
+	client.loop_forever()