Avoid Throwing Exceptions
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 28/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- objective-c
- Área
- authentication, mobile
Línea de trabajo
Comienza leyendo GoogleSignIn/Sources/GIDSignIn.m y siguiendo dónde se exponen las excepciones a los invocadores. Compara ese flujo con el enfoque de NSError y BOOL sugerido en el issue y verifica después que el comportamiento de la biblioteca en el nivel superior informe de los errores sin exponer excepciones a las bibliotecas cliente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem you're having? Please describe.
GoogleSignIn/Sources/GIDSignIn.m is the only place that exposes exceptions. This is done in contrast to the Google style guide which in turn is aligned with Apple's Exception Programming Topics.
Describe the solution you'd like
Instead, error objects should be used: you should take care to catch such exceptions at the top level of the library and translate them into an appropriate return code or state. Clang for instance pass a **NSError and return a BOOL.
Describe alternatives you've considered
N/A
Additional context
I am receiving crashes for Google Sign-In iOS plugin for Flutter, and it seems to be the case for react-native as well. Using exceptions is less informative for the client library, and is bound to become 'unhandled'.
- Lenguaje dominante
- Objective-C
- Estrellas
- 751
- Forks
- 282
- Merge medio
- 2 d 15 h
- PR fusionados (30 d)
- 9
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 google/GoogleSignIn-iOS
-
enhancement triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
google/GoogleSignIn-iOS#621 ·
-
enhancement triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
google/GoogleSignIn-iOS#620 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
google/GoogleSignIn-iOS#440 · 4 comentarios · 2 reacciones ·
-
enhancement triage
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
google/GoogleSignIn-iOS#611 ·
-
Remove Cocoapods Support Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
google/GoogleSignIn-iOS#607 ·
Todos los issues de google/GoogleSignIn-iOS
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
macOS下查找brew::ncurse失败 Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
noah-nuebling/mac-mouse-fix#2054 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
obsproject/obs-studio#13918 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
HigherOrderCO/Bend#842 · 2 comentarios ·