binance / binance/binance-connector-python

SPOT : UserDataStreamEventsResponse does not deserialize correctly, instance is always None

Abierto
#477 3 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Python
Estrellas
2.9k
Forks
697
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

### Description

The `UserDataStreamEventsResponse` model fails to parse user data stream events into their concrete event types (`BalanceUpdate`, `ExecutionReport`, `OutboundAccountPosition`, `EventStreamTerminated`, etc.). All input results in `None` and the event data is completely lost.

### Expected Behavior

When validating a user data stream event, the model should:
1. Parse the event dictionary into the appropriate concrete event type based on the `"e"` field
2. Return a strongly-typed instance (e.g., `BalanceUpdate`, `ExecutionReport`)
3. Preserve all event data fields

Example:
```python
from binance_sdk_spot.websocket_api.models import UserDataStreamEventsResponse

event = {
"e": "balanceUpdate",
"E": 1573200697110,
"a": "BTC",
"d": "100.00000000",
"T": 1573200697068
}

result = UserDataStreamEventsResponse.model_validate(event)
# Expected: result is a BalanceUpdate instance with all fields accessible
# Expected: result.a == "BTC"
# Expected: result.d == "100.00000000"
# Expected: result.E == 1573200697110
# Expected: result.e == "balanceUpdate"
# Expected: result.T == 1573200697068
```

### Actual Behavior

The parsing always fails and returns `None`, losing all event data:

```python
result = UserDataStreamEventsResponse.model_validate(event)
# Actual: result.actual_instance is None
# Actual: All event data is lost
# Actual: Cannot access event fields
```

### Possible Root Cause

The model uses Pydantic's discriminated union pattern but fails to properly discriminate between event types. Issues include:

1. **Schema Validation Failure**: The discriminator field or schema mapping is not correctly configured
2. **Type Resolution Failure**: The model cannot resolve which concrete event type to instantiate
3. **Silent Failure**: Instead of raising validation errors, it silently returns `None`

This is similar to the `SymbolFilters` discriminated union issue that was previously identified, where filter type discrimination failed.

### Impact

- **Complete Data Loss**: All user data stream events are lost when using this model
- **No Balance Updates**: Balance changes from `outboundAccountPosition` events cannot be processed
- **No Order Updates**: Order status changes from `executionReport` events cannot be tracked
- **No Stream Monitoring**: Stream termination events cannot be detected

### Related Issues

- Similar to #476 discriminated union parsing issue

### References

**Affected Version:** >=2.1.0
Related or same as #405

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Empieza con UserDataStreamEventsResponse.model_validate y los modelos de eventos concretos relacionados; después, compara el comportamiento de la unión discriminada con el problema de SymbolFilters en #476 y el problema relacionado #405. Se considera terminado cuando un evento como balanceUpdate produce una instancia de evento tipada y conserva sus campos, en lugar de dejar actual_instance como None.

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

Evaluación

Stack tecnológico
python
Área
api
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.