Repository navigation
Conversation
0ffb9e5 to
3454dd3
Compare
|
Hi @romanbsd, I tested the current head ( I also ran the frozen July server conformance suite ( The bundled client fixture had 270 passing and 44 failing scored checks. It creates The main gap I see is the HTTP path: modern POSTs create a session ID, while I have a local opt-in July candidate with stateless listen and tests. It passed the scored server checks on the same frozen harness, though it is on an older base. I'm happy to help adapt the relevant pieces here instead of opening a competing PR. |
3454dd3 to
215e03b
Compare
Add Discover/listen/per-request `_meta` alongside the legacy initialize era, and close parked listen SSE on cancel so refresh cannot fan-out duplicates or leak HTTP connections.
215e03b to
28a2865
Compare
|
@EhsanAzish80 thanks for the feedback! I fixed the incompatibilities and did some refactoring along the way. |
|
Thanks Roman, I retested One test gate remains: |
|
pushed another commit, is that what you had in mind? |
|
Yes, that’s what I had in mind. Thanks for the quick update! The client test now reaches the July path. |
1c5c531 to
e29d794
Compare
|
more fixes pushed |
|
it's converging :) |
|
Thank you! |
Summary
server/discover, per-request_meta(protocol version / client capabilities), modernresultType, and-32022unsupported-version errors.subscriptions/listenover stateful HTTP SSE (filtered fan-out, no legacy GET mix on modern sessions), with cancel closing the parked stream on both client and server so listen refresh does not leak connections or duplicate notifications.ModernProtocolTests.Test plan
swift test --filter ModernProtocolTestsswift test(full package)discover→subscriptions/listen→ resource subscribe refresh cancels prior listen without duplicateonNotificationinitializesession still works; modern session rejects legacy GET notification stream-32022withdata.requested/data.supported