opengnsys
/
ogclone-engine
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

refs #2608: Fix: avoid crash in _clientip when ip output includes empty entries #105

Merged
nserrano merged 2 commits from ticket-2608 into main 2025-08-08 10:12:38 +02:00
Conversation 0 Commits 2 Files Changed 2
narenas commented 2025-08-08 03:03:51 +02:00
Collaborator

Este PR soluciona un fallo en la función _clientip() (ProtocolLib.py) que provocaba una excepción cuando la salida del comando ip -json address show up contenía elementos vacíos ({}) o mal formateados.

  • Problema detectado
    En ciertos entornos, el comando ip puede devolver objetos vacíos como parte del JSON. Al intentar acceder directamente a e['ifname'] sin verificar que la clave exista, se produce una excepción (KeyError o TypeError), interrumpiendo el flujo de ejecución en llamadas como ogMcastReceiverFile().

  • Pruebas realizadas

  1. Pasados los tet de integración.
Este PR soluciona un fallo en la función _clientip() (ProtocolLib.py) que provocaba una excepción cuando la salida del comando ip -json address show up contenía elementos vacíos ({}) o mal formateados. - Problema detectado En ciertos entornos, el comando ip puede devolver objetos vacíos como parte del JSON. Al intentar acceder directamente a e['ifname'] sin verificar que la clave exista, se produce una excepción (KeyError o TypeError), interrumpiendo el flujo de ejecución en llamadas como ogMcastReceiverFile(). - Pruebas realizadas 1. Pasados los tet de integración.
narenas added 1 commit 2025-08-08 03:03:51 +02:00
narenas changed title from refs #2608: Fix: avoid crash in _clientip when ip output includes empty entries to WIP: refs #2608: Fix: avoid crash in _clientip when ip output includes empty entries 2025-08-08 03:03:54 +02:00
narenas changed title from WIP: refs #2608: Fix: avoid crash in _clientip when ip output includes empty entries to refs #2608: Fix: avoid crash in _clientip when ip output includes empty entries 2025-08-08 03:04:09 +02:00
nserrano was assigned by narenas 2025-08-08 03:04:25 +02:00
nserrano added 1 commit 2025-08-08 10:11:12 +02:00
nserrano merged commit 719cfff000 into main 2025-08-08 10:12:38 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
nserrano
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: opengnsys/ogclone-engine#105
There is no content yet.
Delete Branch "ticket-2608"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?