Open josephktcheung opened 6 years ago
Also fix #40 by adding connectionParams
to context
so that user can use setUpContainer
callback to implement authentication (similar to sample 10)
Any update on this? This is an awesome framework! would be a shame to stop updating it
This is to fix #38. Since there are no
CurrentRequest
,CurrentResponse
anddataLoaders
in the context / container when it's subscription, the resolver will fail when it tries to access those from context inbuildCustomResolvers
.Since
CurrentRequest
andCurrentResponse
can beundefined
in subscriptions, I use the try catch to check if those 2 keys exists in theContainer
.I also set
dataLoaders
to an empty array in context when a new subscriber connects to server.