Skip to content

Commit 7b27770

Browse files
Merge branch 'fix/remove-close-on-complete' into 'master'
Remove websocket close on `complete` message See merge request PassiveLogic/platform/graphqlws!3
2 parents 5ab2866 + ec7c3cc commit 7b27770

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

Sources/GraphQLWS/Server.swift

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ public class Server {
155155
subscribeFuture.whenSuccess { result in
156156
guard let streamOpt = result.stream else {
157157
// API issue - subscribe resolver isn't stream
158-
self.error(.internalAPIStreamIssue(errors: result.errors))
158+
self.sendError(result.errors, id: id)
159159
return
160160
}
161161
let stream = streamOpt as! ObservableSubscriptionEventStream
@@ -177,12 +177,11 @@ public class Server {
177177
onCompleted: { [weak self] in
178178
guard let self = self else { return }
179179
self.sendComplete(id: id)
180-
self.messenger?.close()
181180
}
182181
).disposed(by: self.disposeBag)
183182
}
184183
subscribeFuture.whenFailure { error in
185-
self.error(.graphQLError(error))
184+
self.sendError(error, id: id)
186185
}
187186
}
188187
else {
@@ -205,7 +204,6 @@ public class Server {
205204
self.error(.notInitialized())
206205
return
207206
}
208-
onExit()
209207
}
210208

211209
private func onConnectionTerminate(_: ConnectionTerminateRequest, _ messenger: Messenger) {

0 commit comments

Comments
 (0)