Retain Cycle due to strong reference to PresentingViewController
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 25/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- objective-c
- Ambito
- authentication, mobile-dev
Direzione di ricerca
Inizia esaminando PR #919 e il percorso di inizializzazione di AppAuth che riceve presentingViewController. Confronta la modifica proposta della ownership con il ciclo di vita del singleton segnalato, quindi verifica che il presenting controller possa essere rilasciato senza un retain cycle; il lavoro è completato quando la regressione è coperta e il comportamento esistente dell’SDK rimane invariato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Describe the bug
When initialize AppAuth in singleton and then pass the presentingViewController, it's resulting in retain cycle.
So we have created this PR, which based on our fork that we are using for almost a year or more with no noticeable issue:
https://github.com/openid/AppAuth-iOS/pull/919
- Lingua principale
- Objective-C
- Stelle
- 2k
- Fork
- 866
- Merge medio
- 9g 20h
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi 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 openid/AppAuth-iOS
-
enhancement
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 60/100
openid/AppAuth-iOS#908 ·
-
Custom browser user-agent fails the flow with "Unable to open Safari." while the browser opens (it returns the result of canOpenURL:)Forse già presa @federicobartoli l’ha presa 1 giorno fa. Apertabug triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 25/100
openid/AppAuth-iOS#980 ·
-
Remove Cocoapods SupportAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
openid/AppAuth-iOS#962 ·
-
enhancement triage
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
openid/AppAuth-iOS#953 ·
-
enhancement triage
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
openid/AppAuth-iOS#948 ·
Tutte le issue di openid/AppAuth-iOS
Issue simili
-
s: need triage
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
Tencent-TDS/KuiklyUI#1786 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
DataDog/dd-sdk-ios#3293 ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
callstackincubator/voltra#340 ·
I maintainer di solito rispondono entro 4 giorni
-
Fabric TextInput: prepareForRecycle leaves stale inputAccessoryViewButtonLabel (empty Done bar)ApertaNeeds: Author Feedback Needs: Repro
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
react/react-native#58952 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
iOS: number-pad TextInput shows a toolbar with a "Default" buttonForse già presa @Gauvino l’ha presa 3 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100