binance / binance/binance-connector-python
[Security] WebSocket ssl Parameter Accepts False/None Without Warning
- Lingua principale
- Python
- Stelle
- 2.9k
- Fork
- 697
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
## Bug Name
WebSocket ssl Parameter Accepts False/None Without Warning
## Attack Scenario
The WebSocket init_connection method passes configuration.https_agent directly to aiohttp.ws_connect(ssl=...). When ssl=False, aiohttp disables SSL verification entirely without any warning from the SDK.
## Impact
A user who sets https_agent=False would silently disable TLS certificate verification for all WebSocket connections, enabling MITM attacks.
## Components
File: common/src/binance_common/websocket.py (lines 164-170, 182-188). ssl parameter passed from configuration.https_agent.
## Reproduction
1. Create WebSocket configuration with https_agent=False.
2. Connect to WebSocket API.
3. SSL verification is silently disabled -- no warning or error.
## Fix
Add a validation check: if https_agent is False, raise a warning or error. Document the security implications clearly.
## Details
Finding ID: SEC-05
Severity: Low
---
Researcher: Independent Security Researcher -- Mefai Security Team
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia in common/src/binance_common/websocket.py, nel codice di init_connection intorno alle righe 164-170 e 182-188, quindi segui la configurazione di https_agent fino a aiohttp.ws_connect. Riproduci una connessione con https_agent=False e verifica che la modifica completata non consenta più silenziosamente la verifica SSL disabilitata e che le implicazioni per la sicurezza siano documentate.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- networking, security
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Tranquilla
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 56/100