Commit 4b5aa10d authored by Stéphan Bernard's avatar Stéphan Bernard
Browse files

Merge de la branche MQTT, suppression du passage à l'UTC au moindre problème...

Merge de la branche MQTT, suppression du passage à l'UTC au moindre problème NTP. Il reste du boulot à faire donc appatition d'un TODO.
parent 50939fa5
No related merge requests found
Showing with 55 additions and 3 deletions
+55 -3
...@@ -962,9 +962,8 @@ void setup() ...@@ -962,9 +962,8 @@ void setup()
setInterval(120); /* Try every 2 minutes until success */ setInterval(120); /* Try every 2 minutes until success */
if (!tz.setLocation(timezone)) { if (!tz.setLocation(timezone)) {
Serial.print(MSG[WARN]); Serial.print(MSG[WARN]);
Serial.print(F("Timezone setting failed -> ")); Serial.println(F("Timezone setting failed."));
Serial.println(timezone); /* tz = UTC; */
tz = UTC;
} }
digitalWrite(BLUE_LED, HIGH); /* HIGH value turns it off */ digitalWrite(BLUE_LED, HIGH); /* HIGH value turns it off */
......
# TODO
## Désactivation de MQTT
De même qu'il est possible de désactiver la sauvegarde des mesures
dans des fichiers csv, on doit permettre de désactiver la fonctionnalité
d'envoi des données à un broker MQTT.
Actuellement, donner une adresse bidon pour le broker reste possible, mais
oblige le nœud à tenter d'accéder au broker pendant 10s à chaque mesure.
## Clarifier la page de paramètres Wifi
La saisie de l'IP du broker MQTT a été rajoutéa à cette page,
et le bouton "Set" concerne l'ensemble des paramètres. En réalité,
- une valeur vide pour l'IP du broker laisse le broker MQTT inchangé,
- une valeur vide pour le ssid du point d'accès laisse les paramètres
du client wifi inchangés.
Néanmoins ça n'est ni documenté ni intuitif. On pourrait imaginer deux boutons
set, comme sur la page de la date et l'heure (selon qu'on utilise l'interface
pour régler manuellement la date et l'heure ou pour paramétrer le client NTP).
## Optimiser le code
Il y a beaucoup de choses dans ce code car ce nœud capteur fait beaucoup
trop de choses à la fois :
- Point d'accès Wifi
- Client Wifi (en même temps)
- Client NTP
- Client MQTT
- Serveur http ET https
- Nœud capteur
- Sauvegarde des données capteur dans des fichiers
- le café (ah non, dommage)
- …
Du coup le code est un peu fouills. Il serait bien de faire un peu de relecture
pour le reprendre, le rendre plus clair et l'optimiser.
Quelques pistes :
- regrouper les *handlers* "/sensor", "/mqttbroker", "/ap", "/status", "/dtz",
et "/sensorsetup" en un seul *handler* (car ils n'envoient que des valeurs
qu'on a sous la main, contrairement à "/scan" ou "/listfiles").
- permettre de désactiver en dur (#ifdef SERIAL_OUTPUT) les sorties console
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment