cleanup: Toten Code in handle_contact_message entfernen (issue #5) #9

Merged

ppfeiffer merged 1 commit from fix/dead-code-sender-pubkey into main

2026-03-13 10:43:33 +01:00

ppfeiffer commented

2026-03-13 07:56:45 +01:00

Owner

Problem

In handle_contact_message() wird sender_pubkey in zwei aufeinanderfolgenden Zeilen gesetzt, wobei Zeile 323 sofort durch Zeile 324 überschrieben wird:

# Vorher
sender_pubkey = payload.get('pubkey_prefix', '')  # Zeile 323 – toter Code!
sender_pubkey = sender_id                          # Zeile 324 – überschreibt sofort

Zeile 323 hatte keinen Effekt: der zugewiesene Wert wurde nie gelesen.

Fix

Zeile 323 entfernt, Kommentar auf Zeile 324 präzisiert:

# Nachher
sender_pubkey = sender_id  # Default: pubkey_prefix, replaced below if contact found

Tests

426 Tests grün (2 vorab bereits fehlgeschlagene test_config_validation-Tests sind unabhängig von dieser Änderung).

Schließt #5

## Problem In `handle_contact_message()` wird `sender_pubkey` in zwei aufeinanderfolgenden Zeilen gesetzt, wobei Zeile 323 sofort durch Zeile 324 überschrieben wird: ```python # Vorher sender_pubkey = payload.get('pubkey_prefix', '') # Zeile 323 – toter Code! sender_pubkey = sender_id # Zeile 324 – überschreibt sofort ``` Zeile 323 hatte keinen Effekt: der zugewiesene Wert wurde nie gelesen. ## Fix Zeile 323 entfernt, Kommentar auf Zeile 324 präzisiert: ```python # Nachher sender_pubkey = sender_id # Default: pubkey_prefix, replaced below if contact found ``` ## Tests 426 Tests grün (2 vorab bereits fehlgeschlagene `test_config_validation`-Tests sind unabhängig von dieser Änderung). Schließt #5

ppfeiffer added 1 commit

2026-03-13 07:56:46 +01:00

cleanup: Toten Code in handle_contact_message entfernen

Build and Push Docker Image / build (pull_request) Failing after 41s

Details

Tests / test (pull_request) Failing after 26s

Details

8cf8729fd4

sender_pubkey wurde auf Zeile 323 aus dem Payload gelesen und sofort
auf Zeile 324 mit sender_id überschrieben. Der Payload-Read war damit
toter Code. Die redundante Zeile wird entfernt; der Kommentar erklärt
nun klar die Intention des verbleibenden Defaults.

Schließt #5