query string is not being parsed in middleware handlers about fastify

[mcollina]

You should use the plugin interface, not the middleware one. There is plenty of docs in the repo, so there should be all that you need.
All the requirements you are listing can be implemented using the normal fastify API.

Have a look and let us know if you need any specifics details.

I'd seen this start over at Apollo (apollographql/apollo-server#402, apollographql/apollo-server#421). I assume they'll include it eventually since they have an option for micro.

But I wanted to port a project from work that runs with Apollo and express over to fastify. I like the possibilities with hooks for some use cases better than clunky express (authn, authz, graphql context, etc.). mcollina mentioned it as a plugin, so I took a stab at my first OSS project and made fastify-apollo. I hope an Apollo server lines up with fastify's ecosystem.

[delvedor]

Hi! Yes it's the expected behavior as you can see here.
The parsing is done after the routing. If you want you can use the preHandler hook, that gives you everything you need :)

[mcollina]

I think you should not use middlewares for integrating with Apollo, but rather develop a full blown plugin for Fastify.

[phra]

i can investigate also that way, for now i've just followed the structure of other integrations.
can you please give me additional infos on the solution you are thinking about?

as i can see the requirements to make apollo works are:

• ability to intercept only certain paths (/graphql and /graphiql)
• ability to access to the req.body for POST or req.query for GET
• ability to set custom headers on the Reply
• ability to write raw data in the Reply stream
• ability to close the Reply stream

you can also find the actual example usage with the middleware solution here

thanks.

EDIT: that's the original issue that i opened 20 days ago, if you want to add a comment there for some reason.

[phra]

ok, i'll let you know guys. thanks for the suggestion.

[delvedor]

Closing this, feel free to reopen if you have some issues :)

[mcollina]

@coopnd this is amazing work! Would you like to send us a PR to the README https://github.com/fastify/fastify#ecosystem

Let us know if you get any bugs in fastify, open them or send PRs.

[delvedor]

Awesome, thanks! :)

[phra]

@coopnd good job! 🚀