Check `NetworkRequest#mDestroyed` before each network request?
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
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.
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
- Lingua principale
- Java
- Stelle
- 74
- Fork
- 36
- Merge medio
- 1g 6h
- PR unite (30g)
- 15
Preparare l'ambiente
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.
Altre issue di nextcloud/Android-SingleSignOn
-
0. Needs triage bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
nextcloud/Android-SingleSignOn#1081 ·
I maintainer di solito rispondono entro 1 giorno
-
0. Needs triage bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
nextcloud/Android-SingleSignOn#894 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: Brotli or ZSTD reverse proxy compression breaks SSO authentication in some android appsAperta0. Needs triage bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
nextcloud/Android-SingleSignOn#859 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
0. Needs triage enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
nextcloud/Android-SingleSignOn#849 ·
I maintainer di solito rispondono entro 1 giorno
-
0. Needs triage enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
nextcloud/Android-SingleSignOn#771 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di nextcloud/Android-SingleSignOn
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
oracle/javavscode#652 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
OpenAPITools/openapi-generator#25014 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
AloisSeckar/demos-java#380 ·