[2.8] Stop clobbering node identity from 2.8 feeder NodeDB data

Abierto
#445 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
Medio día
Aptitud para principiantes
42/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
python
Área
backend

Línea de trabajo

Start with the node upsert in packets/serializers.py around lines 1386 and 1528-1543, then the int/str comparison at packets/services/node_info.py line 17. TypeConversions.cpp (lines 150-151) is firmware-side context, not code in this repo. Done means 2.8 NodeDB fixtures keep the stored MAC, name and public key, with a unit test for each of the four rules.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

enhancement meshtastic-2.8

Part of pskillen/meshflow-api#437 (Meshtastic 2.8.x support). Code: ID-8 · Priority: Must · Size: S

Summary

2.8 feeders hand the bot NodeDB entries with zero-filled macaddr (TypeConversions.cpp:150-151) and long names truncated to 24 bytes. Our node upsert (packets/serializers.py:1386, 1528-1543) overwrites good data with these, destroying the MAC evidence ID-2 needs and flip-flopping names.

  • Ignore blank / all-zero MAC.
  • Never replace a name with its own ≤24-byte prefix.
  • Never clear a stored public key.
  • Prefer values from over-the-air NodeInfo packets over feeder NodeDB upserts.
  • Fix latent int/str comparison bug at packets/services/node_info.py:17.

Acceptance criteria

  • 2.8 NodeDB fixtures leave MAC, name and key intact
  • Unit tests for each rule

Depends on: none

Lenguaje dominante
Python
Estrellas
1
Forks
0
Merge medio
1 d 20 h
PR fusionados (30 d)
5

Preparar el entorno

  • Incluye un Dockerfile o un archivo de Docker Compose
  • Tiene una plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de pskillen/meshflow-api

Todos los issues de pskillen/meshflow-api

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.