Subscriptions don't call `onUnsubscribe` if a socket is unexpectedly closed
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 45/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Swift
- Sterne
- 182
- Forks
- 29
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus swift-server/RediStack
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 55/100
swift-server/RediStack#135 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
swift-server/RediStack#133 · 5 Kommentare ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
swift-server/RediStack#131 ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
swift-server/RediStack#129 · 1 Kommentar ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
swift-server/RediStack#127 ·
Alle Issues in swift-server/RediStack
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
swiftlang/swift-format#1312 ·
-
LiquidGlass / LiquidGlassContainer (iOS): padded content measured at the outer width and overflows by the paddingEvtl. vergeben @aleclarson hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
NativeScript/NativeScript#11561 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 65/100
omacom/try-omarchy#402 ·
Maintainer antworten meist innerhalb von 1 Tag
-
area: build-and-ci S3: minor
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
manaflow-ai/cmux#22595 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag