nextcloud / nextcloud/Android-SingleSignOn
Check `NetworkRequest#mDestroyed` before each network request?
Nessuno ha ancora preso questa issue.
- Lingua principale
- Java
- Stelle
- 74
- Fork
- 36
- Merge medio
- 1g 19h
- PR unite (30g)
- 17
Descrizione
I recently ran into an issue while trying to make a network request on an NextcloudAPI instance, on which I called close() before.
Of course, it failed, but at an odd place, that I didn't expected (I think context was null at some place).
This made me wondering why we do check for NetworkRequest#mDestroyed when connecting, but not before each request. Wouldn't make it sense to add this check also at the beginning of AidlNetworkRequest#performNetworkRequestV2 and throw a more meaningful IllegalStateException or something alike?
cc @David-Development
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Parti da AidlNetworkRequest#performNetworkRequestV2 e confrontalo con il controllo esistente di NetworkRequest#mDestroyed eseguito durante connect. Verifica che la chiamata a close() prima di una richiesta venga rilevata all’inizio dell’operazione e produca una IllegalStateException significativa anziché un successivo errore dovuto a un contesto null.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- android, java
- Ambito
- mobile
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Attiva
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 72/100