David Glasser
David Glasser
Thanks, I'll try to make time to review this when I have time to refresh my memory of how Hapi works. I'm not super clear on how this fixes #4967...
Thanks. Also don't forget to rebase when you do so (I think the conflicts are just spelling fixes.)
Would it make more sense to let you override `cacheKeyString` instead? (Note that this would also subsume the functionality of `extraCacheKeyData` which could be deprecated.)
I think I might not have been clear here. I feel like the concept of "dynamic prefix" is too specific. Both "dynamic prefix" and "extra cache key data" are ways...
This seems valuable but I'm going to remove it from AS3 blockers because I don't think fixing this needs to be a breaking change: we can start passing context as...
The challenge is that extensions are readonly on the core GraphQLError, not something that's part of apollo-server... We could I suppose provide a mechanism for letting you replace specific errors...
@IvanGoncharov I wonder if you have any thoughts on this? I know you've been thinking about how to simplify `apollo-server-errors`.
@simlu Part of the challenge here is that formatError can be called both from within the "request pipeline" once the context is set up... *or* earlier because there's a problem...
Apollo Server 3 does not support subscriptions. I'm not sure it makes much sense to add partial support like this. Fastify doesn't provide any mechanism to let you call the...
We answered this question in https://github.com/apollographql/apollo-server/issues/6020#issuecomment-1020543875