Skip to content

Commit 50dc3a0

Browse files
author
Jay Herron
committed
Adds subscription resolver nil handling
1 parent a52bed8 commit 50dc3a0

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

Sources/GraphQL/Subscription/Subscribe.swift

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -255,8 +255,14 @@ func executeSubscription(
255255
return SourceEventStreamResult.failure(error)
256256
} else if let observable = resolved as? Observable<Any> {
257257
return SourceEventStreamResult.success(observable)
258+
} else if resolved == nil {
259+
return SourceEventStreamResult.failure(
260+
GraphQLError(message: "Resolved subscription was nil")
261+
)
258262
} else {
259-
return SourceEventStreamResult.failure(GraphQLError(message: "Subscription field resolver must return an Observable<Any>"))
263+
return SourceEventStreamResult.failure(
264+
GraphQLError(message: "Subscription field resolver must return an Observable<Any>, not \(Swift.type(of:resolved))")
265+
)
260266
}
261267
}
262268
}

0 commit comments

Comments
 (0)