Proposal: Restify support about apollo-server HOT 11 CLOSED

@helfer yes, sorry, forgot a bit about it! I have a lot to do currently (mainly client stuff, that's why the PRs are going there... 😉 ) but yes, I still plan doing that at some point! I'll tell you then when I do it if you can help me!

from apollo-server.

@rricard Yeah, that's a great idea. If you add tests, that will also be a great way to point to concrete examples for how to use it.

from apollo-server.

@helfer I've got great news! I finally tried to run apollo-server on restify: it works perfectly 🎉 . So the idea now is to write tests to ensure we don't break compat with restify, I'll work on that myself. I'll also write some docs on how to integrate w/ restify!

from apollo-server.

@rricard Yes, I'm definitely for adding support for Restify! But at the moment I'm having a bit of trouble seeing how you would integrate it tightly with Apollo Server? Would you just use it as an alternative to Express, HAPI, Koa, etc?

Is the idea to essentially use Restify as a backend and automatically generate resolvers or a schema somehow?

If you have an example for how you integrated express-graphql in Restify and what you would like the integration to look like in an ideal world, that would be great!

from apollo-server.

The main issue with restify is it can't live with express in the same codebase since both override stuff globally. We need our graphql schema in the same codebase ...

Right now it looks like that:

const graphqlHTTP = require('express-graphql');

const graphQLEndpoint = graphqlHTTP(...);

app.get('/graphql', graphQLEndpoint);
app.post('/graphql', graphQLEndpoint);

It works but I think we could do better!

from apollo-server.

For instance we could define the graphql endpoint as a middleware. I'll try to think of a correct API and propose that to you!

from apollo-server.

@rricard can't you use Apollo Server the same way? http://docs.apollostack.com/apollo-server/setup.html#apolloExpress

from apollo-server.

Last time I checked, it failed with some restify internals (not express graphql). At least if we want to go that way and use restify via the express support, tests should be added to ensure restify is not broken. I should start with that...

from apollo-server.

@rricard are you still interested in this? I think it would be pretty useful! Let me know if I can help in any way.

from apollo-server.

Closing in favor of #125

from apollo-server.

Thank for looking into this!

from apollo-server.