GIDSignIn.sharedInstance.restorePreviousSignIn() sometimes fails to invoke completion handler
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- objective-c
- Ambito
- authentication, mobile
Direzione di ricerca
Inizia con restorePreviousSignIn() nell’app di esempio DaysUntilBirthday e riproduci il problema usando le modifiche collegate che lo invocano due volte. Esegui il caso di aggiornamento del token regolando kMinimalTimeToExpire e kMinimumRestoredAccessTokenTimeToExpire come descritto, quindi traccia il flusso di ripristino. Il lavoro è completato quando entrambi i blocchi di completamento vengono invocati nello scenario riprodotto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Describe the bug
GIDSignIn.sharedInstance.restorePreviousSignIn() does not always invoke the completion handler.
I suspect these two closed issues may have the same root cause:
- https://github.com/google/GoogleSignIn-iOS/issues/360
- https://github.com/google/GoogleSignIn-iOS/issues/378
To Reproduce
There appears to be two requirements to reproduce this:
- restorePreviousSignIn() must be invoked multiple times before the first call completes
- the tokens must be close enough to expiring to trigger refresh logic
Specific repro steps:
- Make local changes to DaysUntilBirthday sample app such that restorePreviousSignIn() is invoked twice - see:
https://github.com/google/GoogleSignIn-iOS/commit/01d8cb32d1315cea141feeed5c055c9beb912561 - Sign in
- Kill and restart the app. Check console: should see output below, indicating that both completion blocks are invoked:
0: Restoring
1: Restoring
0: Done: Optional(<GIDGoogleUser: 0x600002124820>), nil
1: Done: Optional(<GIDGoogleUser: 0x600002124820>), nil - Kill the app and wait for about an hour. I was also able to reproduce immediately by changing kMinimalTimeToExpire and kMinimumRestoredAccessTokenTimeToExpire to both be 60.0*59.9.
- Restart the app and check console output.
Expected behavior
console output should be the following, indicating that both completion blocks were invoked:
0: Restoring
1: Restoring
0: Done: Optional(<GIDGoogleUser: 0x600002124820>), nil
1: Done: Optional(<GIDGoogleUser: 0x600002124820>), nil
Actual behavior
console output is the following, indicating the first completion block was not invoked:
0: Restoring
1: Restoring
1: Done: Optional(<GIDGoogleUser: 0x600002124820>), nil
- Lingua principale
- Objective-C
- Stelle
- 751
- Fork
- 282
- Merge medio
- 4g 20h
- PR unite (30g)
- 11
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.
Altre issue di google/GoogleSignIn-iOS
-
enhancement triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
google/GoogleSignIn-iOS#621 ·
-
enhancement triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
google/GoogleSignIn-iOS#620 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
google/GoogleSignIn-iOS#440 · 4 commenti · 2 reazioni ·
-
enhancement triage
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
google/GoogleSignIn-iOS#611 ·
-
Remove Cocoapods Support Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
google/GoogleSignIn-iOS#607 ·
Tutte le issue di google/GoogleSignIn-iOS
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
objective-see/LuLu#925 ·
-
macOS下查找brew::ncurse失败 Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
noah-nuebling/mac-mouse-fix#2054 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
obsproject/obs-studio#13918 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
HigherOrderCO/Bend#842 · 2 commenti ·