Avoid Throwing Exceptions
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 28/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- objective-c
- Domaine
- authentication, mobile
Piste de recherche
Commencez par lire GoogleSignIn/Sources/GIDSignIn.m et suivre les endroits où les exceptions sont exposées aux appelants. Comparez ce flux avec l’approche NSError et BOOL suggérée par l’issue, puis vérifiez que le comportement de la bibliothèque au niveau supérieur signale les erreurs sans exposer d’exceptions aux bibliothèques clientes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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'.
- Langage dominant
- Objective-C
- Étoiles
- 756
- Forks
- 283
- Merge moyen
- 3 j 1 h
- PR mergées (30 j)
- 2
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de google/GoogleSignIn-iOS
-
Revoke request uses GET instead of POSTPeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverteenhancement triage
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
google/GoogleSignIn-iOS#621 ·
-
enhancement triage
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
google/GoogleSignIn-iOS#620 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
google/GoogleSignIn-iOS#440 · 4 commentaires · 2 réactions ·
-
bug triage
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
google/GoogleSignIn-iOS#647 ·
-
enhancement triage
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
google/GoogleSignIn-iOS#611 ·
Toutes les issues de google/GoogleSignIn-iOS
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
sparkle-project/Sparkle#2932 ·
Les mainteneurs répondent en général sous 1 jour
-
Typo in installer: “addition protections” → “additional protections”Peut-être pris @Jah-yee l’a pris il y a 1 jour. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 94/100
objective-see/BlockBlock#111 ·
-
s: need triage
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
Tencent-TDS/KuiklyUI#1786 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
DataDog/dd-sdk-ios#3293 ·
Les mainteneurs répondent en général sous 3 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
callstackincubator/voltra#340 ·
Les mainteneurs répondent en général sous 4 jours