Subscriptions don't call `onUnsubscribe` if a socket is unexpectedly closed
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Start by reading the subscription cancellation and socket error handling paths to find where onUnsubscribe is called and how unexpected closure is handled. The issue gives no file or test names, so trace those paths from the subscription entry point; done means onUnsubscribe is called when the socket closes unexpectedly as well as on formal cancellation.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
onUnsubscribe is currently only called when a subscription is formally cancelled by the user. If a socket error closes the connection, the subscription isn't alive anymore, but because onUnsubscribe isn't triggered there's no good way of knowing that this event was omitted.
I've currently worked around it using ARC retain/release calls. On deinit of a reference type, the onUnsubscribe is implied, though I'm not even certain that this works in practice.
- Langage dominant
- Swift
- Étoiles
- 182
- Forks
- 29
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un 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 swift-server/RediStack
-
Difficulté 2/5 1-3 heures Accessibilité débutants 55/100
swift-server/RediStack#135 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
swift-server/RediStack#133 · 5 commentaires ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
swift-server/RediStack#131 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
swift-server/RediStack#129 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
swift-server/RediStack#127 ·
Toutes les issues de swift-server/RediStack
Issues similaires
-
desktop-app
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
raullenchai/Rapid-MLX#4975 ·
Les mainteneurs répondent en général sous 1 jour
-
[iPhone Duo] Address bar in Duo folded in portrait orientation missing leading and trailing paddingOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
mozilla-mobile/firefox-ios#36179 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 69/100
cuappdev/resell-ios#131 ·
-
📦 expo-modules-core 📦 expo-updates contributor: external iOS needs review sdk-58 severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
expo/expo#51852 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug: Swift integration guide uses inconsistent client variable namesPeut-être pris @timokoethe l’a pris aujourd’hui. Ouvertebug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
logto-io/logto#9787 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour