makeExecutableSchema leads to strange directive behavior

[morleytatro]

Perhaps this is user error, but I noticed that running the following query when using makeExecutableSchema, the error coming back is connected to the BookInput type.

mutation {
    createAuthor(input: {
        name: "M"
    })
}

Here's a repo both with and without makeExecutableSchema used:
https://github.com/morleytatro/graphql-tools-bug

Notice the difference in results.

[yaacovCR]

That particular directive as implemented is no longer supported as of v5, see #842, in particular #842 (comment), #842 (comment), and #842 (comment)

The last comment lists a few new recent alternatives for directive driven validation.

The discrepancy you are seeing is that Apollo is still internally using v4.0.7.

See also #1234 to track a new functional approach to schema directives.

[yaacovCR]

Maybe we can restore support now that all non-legacy functionality does not actually use healSchema/visitSchema/visitSchemaDirectives, just by "unfixing" healSchema.

[tafelnl]

I am not sure if I quite understand it correctly.

But the following code:

const schema = makeExecutableSchema({
  typeDefs,
  resolvers,
  schemaDirectives: {
    isAuth: IsAuthDirective,
  },
});

is not working with "graphql-tools": "^5.0.0". As in, that isAuth directive is not called when querying something.

However when I install "graphql-tools": "^4.0.7", the directive is called.

If this is intended, I think it is quite strange that the official Apollo docs are still suggesting to use this approach: https://www.apollographql.com/docs/graphql-tools/schema-directives/

Even more confusing is that when theIsAuthDirective is not correctly implemented graphql still throws errors. Suggesting that it will work when you make those errors vanish by fixing them.

[tafelnl]

Ah so I found out that there is another doc that state the new method I guess.

Very confusing, but I think this is the problem:

In this doc:
https://www.apollographql.com/docs/graphql-tools/schema-directives/
The suggest the use of makeExecutableSchema. Which is deprecated as far as I understand the above conversation?

But in this doc: https://www.apollographql.com/docs/apollo-server/schema/directives/ (which I guess is newer) they suggest the following method, which I guess is still supported somehow:

const server = new ApolloServer({
  typeDefs,
  resolvers,
  schemaDirectives: {
    upper: UpperCaseDirective,
  }
});

[yaacovCR]

There is a regression in v5 that causes certain directives to no longer work. The examples should all still work as we have tests that ensure they still work. We plan on restoring support for all directives that worked in v4 for the next release. The next release will also suggest a new functional approach, but it should have better backwards compatibility as well.

[yaacovCR]

Apollo is on v4 so they had no changes.

[tafelnl]

I am pretty sure something is up with the combination of makeExecutableSchema on an apollo server with directives. Because when I just shove the typeDefs resolvers and schemaDirectives directly into the ApolloServer options it works perfectly fine.

But I have already been debugging for like hours and hours, so I will not further investigate this problem since the above mentioned solution works for me.

Hope it will help anyone encountering issues and finding this on google

[yaacovCR]

Apollo is using v4, if you use graphql-tools u are using v5 with some directives no longer supported. If you post your directive I can try to see why it is no longer supported, but in reality we are attempting to restore support, with progress tracked in this issue.

I apologize for you encountering this issue. Hopefully this will be resolved soon.