Check `NetworkRequest#mDestroyed` before each network request?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
Línea de trabajo
Comienza en AidlNetworkRequest#performNetworkRequestV2 y compáralo con la comprobación existente de NetworkRequest#mDestroyed que se realiza durante connect. Verifica que llamar a close() antes de una solicitud se detecte al principio de la operación y dé como resultado una IllegalStateException significativa en lugar de un fallo posterior de contexto nulo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Java
- Estrellas
- 74
- Forks
- 36
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 17
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nextcloud/Android-SingleSignOn
-
0. Needs triage bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
nextcloud/Android-SingleSignOn#894 · 1 reacción ·
-
[Bug]: Brotli or ZSTD reverse proxy compression breaks SSO authentication in some android apps Abierto0. Needs triage bug
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
nextcloud/Android-SingleSignOn#859 · 3 comentarios ·
-
0. Needs triage enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
0. Needs triage enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
nextcloud/Android-SingleSignOn#771 · 3 comentarios ·
-
Dependency Dashboard Abiertodependencies overview
Dificultad 3/5 1-2 días Aptitud para principiantes 15/100
Todos los issues de nextcloud/Android-SingleSignOn
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
bug needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 94/100
objectionary/hone-maven-plugin#1061 ·
-
type:bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100