Fix bug that middleware not work when not using default prefix and default schema

[ghost]

Follow Issue #332

I'm not sure I should change the routes file or controller.
But I think that there will be only one schema per request, so I just change the routes.php.

[coveralls]

Coverage remained the same at 70.537% when pulling 2cdcec3 on BinotaLIU:patch-fix-332 into 171631a on Folkloreatelier:develop.

[mfn]

This change doens't look right. prefix is working fine for me.

Can you elaborate more what your problem is and what it does fix?

[ghost]

@mfn as I mentioned in #332, if you changed the prefix, the middleware will not work.

A simple instruction to reproduce that:

1. change the prefix to something else like _
2. create a Query test, which only return Boolean(true).
3. add \Illuminate\Foundation\Http\Middleware\CheckForMaintenanceMode middleware in config('graphql.middelware_schema.default')
4. run artisan down
5. run graphql request /_/default with query

   query test {
     test
   }

Expected result is an HTTP 503 error, but the middleware will not work, so we got the result we wrote in the graphql query, which is { test: true }

[MNF]

@binotaliu , you misspelled the name and should address the comment to @mfn

[ghost]

@MNF sorry for disturb you. i have edited my comment. you can click unsubscribe from the right bar to unsubscribe from future updates.