mirror of
https://github.com/hasura/graphql-engine.git
synced 2024-12-20 22:11:45 +03:00
8ccf7724ce
The code that builds the GraphQL schema, and `buildGQLContext` in particular, is partial: not every value of `(ServerConfigCtx, GraphQLQueryType, SourceCache, HashMap RemoteSchemaName (RemoteSchemaCtx, MetadataObject), ActionCache, AnnotatedCustomTypes)` results in a valid GraphQL schema. When it fails, we want to be able to return better error messages than we currently do. The key thing that is missing is a way to trace back GraphQL type information to their origin from the Hasura metadata. Currently, we have a number of correctness checks of our GraphQL schema. But these correctness checks only have access to pure GraphQL type information, and hence can only report errors in terms of that. Possibly the worst is the "conflicting definitions" error, which, in practice, can only be debugged by Hasura engineers. This is terrible DX for customers. This PR allows us to print better error messages, by adding a field to the `Definition` type that traces the GraphQL type to its origin in the metadata. So the idea is simple: just add `MetadataObjId`, or `Maybe` that, or some other sum type of that, to `Definition`. However, we want to avoid having to import a `Hasura.RQL` module from `Hasura.GraphQL.Parser`. So we instead define this additional field of `Definition` through a new type parameter, which is threaded through in `Hasura.GraphQL.Parser`. We then define type synonyms in `Hasura.GraphQL.Schema.Parser` that fill in this type parameter, so that it is not visible for the majority of the codebase. The idea of associating metadata information to `Definition`s really comes to fruition when combined with hasura/graphql-engine-mono#4517. Their combination would allow us to use the API of fatal errors (just like the current `MonadError QErr`) to report _inconsistencies_ in the metadata. Such inconsistencies are then _automatically_ ignored. So no ad-hoc decisions need to be made on how to cut out inconsistent metadata from the GraphQL schema. This will allow us to report much better errors, as well as improve the likelihood of a successful HGE startup. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4770 Co-authored-by: Samir Talwar <47582+SamirTalwar@users.noreply.github.com> GitOrigin-RevId: 728402b0cae83ae8e83463a826ceeb609001acae
400 lines
16 KiB
Haskell
400 lines
16 KiB
Haskell
{-# LANGUAGE ViewPatterns #-}
|
||
|
||
-- | Defines the 'Parser' type and its primitive combinators.
|
||
module Hasura.GraphQL.Parser.Internal.Input
|
||
( InputFieldsParser (..),
|
||
enum,
|
||
field,
|
||
fieldOptional,
|
||
fieldWithDefault,
|
||
inputParserInput,
|
||
list,
|
||
object,
|
||
pInputParser,
|
||
)
|
||
where
|
||
|
||
import Control.Lens hiding (enum, index)
|
||
import Data.Aeson qualified as A
|
||
import Data.Aeson.Key qualified as K
|
||
import Data.Aeson.KeyMap qualified as KM
|
||
import Data.HashMap.Strict.Extended qualified as M
|
||
import Data.HashSet qualified as S
|
||
import Data.Parser.JSONPath
|
||
import Data.Text.Extended
|
||
import Data.Type.Equality
|
||
import Hasura.GraphQL.Parser.Class.Parse
|
||
import Hasura.GraphQL.Parser.Internal.TypeChecking
|
||
import Hasura.GraphQL.Parser.Internal.Types
|
||
import Hasura.GraphQL.Parser.Schema
|
||
import Hasura.GraphQL.Parser.Variable
|
||
import Hasura.Prelude
|
||
import Hasura.Server.Utils (englishList)
|
||
import Language.GraphQL.Draft.Syntax hiding (Definition)
|
||
|
||
-- ure that out on its own, so we have to be explicit to give
|
||
-- it a little help.
|
||
inputParserInput :: forall k. 'Input <: k => ParserInput k :~: InputValue Variable
|
||
inputParserInput = case subKind @'Input @k of KRefl -> Refl; KBoth -> Refl
|
||
|
||
pInputParser :: forall origin k m a. 'Input <: k => Parser origin k m a -> InputValue Variable -> m a
|
||
pInputParser = gcastWith (inputParserInput @k) pParser
|
||
|
||
-- | Parses some collection of input fields. Build an 'InputFieldsParser' using
|
||
-- 'field', 'fieldWithDefault', or 'fieldOptional', combine several together
|
||
-- with the 'Applicative' instance, and finish it off using 'object' to turn it
|
||
-- into a 'Parser'.
|
||
data InputFieldsParser origin m a = InputFieldsParser
|
||
-- Note: this is isomorphic to
|
||
-- Compose ((,) [Definition (FieldInfo k)])
|
||
-- (ReaderT (HashMap Name (FieldInput k)) m) a
|
||
-- but working with that type sucks.
|
||
{ ifDefinitions :: [Definition origin (InputFieldInfo origin)],
|
||
ifParser :: HashMap Name (InputValue Variable) -> m a
|
||
}
|
||
deriving (Functor)
|
||
|
||
instance Applicative m => Applicative (InputFieldsParser origin m) where
|
||
pure v = InputFieldsParser [] (const $ pure v)
|
||
a <*> b =
|
||
InputFieldsParser
|
||
(ifDefinitions a <> ifDefinitions b)
|
||
(liftA2 (<*>) (ifParser a) (ifParser b))
|
||
|
||
{- Note [When are fields optional?]
|
||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
|
||
|
||
When is an input field (i.e. an argument of a selection set field, or a field of
|
||
an input object) required to be specified? In fact, the GraphQL specification is
|
||
contradictory! As part of the discussion on what
|
||
"[Non-Null](http://spec.graphql.org/June2018/#sec-Type-System.Non-Null)" means,
|
||
it says:
|
||
|
||
> Inputs (such as field arguments), are always optional by default. However a
|
||
> non‐null input type is required. In addition to not accepting the value
|
||
> **null**, it also does not accept omission. For the sake of simplicity
|
||
> nullable types are always optional and non‐null types are always required.
|
||
|
||
However, under the [validity rules for field
|
||
arguments](http://spec.graphql.org/June2018/#sec-Required-Arguments), it says:
|
||
|
||
> Arguments can be required. An argument is required if the argument type is
|
||
> non‐null and does not have a default value. Otherwise, the argument is
|
||
> optional.
|
||
|
||
And under the [validity rules for input object
|
||
fields](http://spec.graphql.org/June2018/#sec-Input-Object-Required-Fields), it
|
||
says:
|
||
|
||
> Input object fields may be required. Much like a field may have required
|
||
> arguments, an input object may have required fields. An input field is
|
||
> required if it has a non‐null type and does not have a default
|
||
> value. Otherwise, the input object field is optional.
|
||
|
||
The first quote above is probably incorrect. Null values were introduced in
|
||
graphql/graphql-spec@3ce8b790da33f52f9258929258877a0a7768c620, when default
|
||
values already existed, and in particular the first quote above was already
|
||
written. Nullable types and values introduced ambiguity. There was an attempt to
|
||
resolve this ambiguity in
|
||
graphql/graphql-spec@0356f0cd105ca54cbdf5eb0f37da589eeac8c641, which introduced
|
||
the last two quotes above. However, clearly some ambiguity was left behind in
|
||
the spec. -}
|
||
|
||
{- Note [The value of omitted fields]
|
||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
|
||
|
||
Suppose we had a table type like this:
|
||
|
||
type article {
|
||
comments(limit: Int!): [comment!]!
|
||
}
|
||
|
||
Since the type of `limit` is non-nullable, and it does not have a default value, it is
|
||
/illegal/ to omit the argument. You’d /always/ have to provide some value---and
|
||
that isn't what we want, because the row limit should be optional. Hence, we
|
||
want to make the argument nullable:
|
||
|
||
type article {
|
||
comments(limit: Int): [comment!]!
|
||
}
|
||
|
||
But this feels questionable. Should we really accept `null` values for `limit`?
|
||
That is, should this query be legal?
|
||
|
||
{
|
||
articles {
|
||
comments(limit: null) { ... }
|
||
}
|
||
}
|
||
|
||
A tempting answer to that question is "yes": we can just treat a `null` value
|
||
for any optional field as precisely equivalent to leaving the field off
|
||
entirely. That is, any field with no default value really just has a default
|
||
value of `null`. Unfortunately, this approach turns out to be a really bad idea.
|
||
It's all too easy to write something like
|
||
|
||
mutation delete_article_by_id($article_id: Int) {
|
||
delete_articles(where: {id: {eq: $article_id}})
|
||
}
|
||
|
||
then accidentally misspell `article_id` in the variables payload, and now you've
|
||
deleted all the articles in your database. Very bad.
|
||
|
||
So we'd really like to be able to have a way to say "this field is optional, but
|
||
`null` is not a legal value," but at first it seems like the GraphQL spec ties
|
||
our hands. Fortunately, there is a way out. The spec explicitly permits
|
||
distinguishing between the following two situations:
|
||
|
||
comments { ... }
|
||
comments(limit: null) { ... }
|
||
|
||
That is, the spec allows implementations to behave differently depending on
|
||
whether an argument was omitted or whether its value was `null`. This is spelled
|
||
out in a few different places in the spec, but §3.10 Input Objects
|
||
<http://spec.graphql.org/June2018/#sec-Input-Objects> is the most explicit:
|
||
|
||
> If the value `null` was provided for an input object field, and the field's
|
||
> type is not a non‐null type, an entry in the coerced unordered map is given
|
||
> the value `null`. In other words, there is a semantic difference between the
|
||
> explicitly provided value `null` versus having not provided a value.
|
||
|
||
Note that this is only allowed for fields that don't have any default value! If
|
||
the field were declared with an explicit `null` default value, like
|
||
|
||
type article {
|
||
comments(limit: Int = null): [comment!]!
|
||
}
|
||
|
||
then it would not be legal to distinguish the two cases. Yes, this is all
|
||
terribly subtle.
|
||
|
||
Okay. So armed with that knowledge, what do we do about it? We offer three
|
||
different combinators for parsing input fields:
|
||
|
||
1. `field` — Defines a field with no default value. The field's nullability is
|
||
taken directly from the nullability of the field's value parser.
|
||
2. `fieldOptional` — Defines a field with no default value that is always
|
||
nullable. Returns Nothing if (and only if!) the field is omitted.
|
||
3. `fieldWithDefault` — Defines a field with a default value.
|
||
|
||
The last of the three, `fieldWithDefault`, is actually the simplest. It
|
||
corresponds to a field with a default value, and the underlying value parser
|
||
will /always/ be called. If the field is omitted, the value parser is called
|
||
with the default value. (This makes it impossible to distinguish omitted fields
|
||
from those explicitly passed the default value, as mandated by the spec.) Use
|
||
`fieldWithDefault` for any field or argument with a non-`null` default value.
|
||
|
||
`field` is also fairly straightforward. It always calls its value parser, so if
|
||
the field is omitted, it calls it with a value of `null`. Notably, there is no
|
||
special handling for non-nullable fields, since the underlying parser will raise
|
||
an error in that case, anyway. Use `field` for required fields, and combine
|
||
`field` with `nullable` for optional fields with a default value of `null`.
|
||
|
||
`fieldOptional` is the most interesting. Unlike `field` and `fieldWithDefault`,
|
||
`fieldOptional` does not call its underlying value parser if the field is not
|
||
provided; it simply returns Nothing. If a value /is/ provided, it is passed
|
||
along without modification. This yields an interesting interaction when the
|
||
value parser does not actually accept nulls, such as a parser like this:
|
||
|
||
fieldOptional $$(litName "limit") Nothing int
|
||
|
||
This corresponds to the `limit` field from our original example. If the field is
|
||
omitted, the `int` parser is not called, and the field parser just returns
|
||
Nothing. But if a value of `null` is explicitly provided, it is forwarded to the
|
||
`int` parser, which then rejects it with a parse error, since it does not accept
|
||
nulls. This is exactly the behavior we want.
|
||
|
||
This semantics can appear confusing. We end up with a field with a nullable type
|
||
for which `null` is not a legal value! A strange interpretation of “nullable”,
|
||
indeed. But realize that the nullability really means “optional”, and the
|
||
behavior makes more sense.
|
||
|
||
As a final point, note that similar behavior can be obtained with
|
||
`fieldWithDefault`. The following creates a boolean field that defaults to
|
||
`false` and rejects `null` values:
|
||
|
||
fieldWithDefault $$(litName "includeDeprecated") Nothing (VBoolean False) boolean
|
||
|
||
This is a perfectly reasonable thing to do for exactly the same rationale behind
|
||
the use of `fieldOptional` above. -}
|
||
|
||
-- | Creates a parser for an input field. For nullable value parsers, if an
|
||
-- input field is omitted, the provided parser /will be called/ with a value of
|
||
-- `null`.
|
||
field ::
|
||
(MonadParse m, 'Input <: k) =>
|
||
Name ->
|
||
Maybe Description ->
|
||
Parser origin k m a ->
|
||
InputFieldsParser origin m a
|
||
field name description parser =
|
||
InputFieldsParser
|
||
{ ifDefinitions = [Definition name description Nothing $ InputFieldInfo (pType parser) Nothing],
|
||
ifParser = \values -> withKey (Key (K.fromText (unName name))) do
|
||
value <-
|
||
onNothing (M.lookup name values <|> nullableDefault) $
|
||
parseError ("missing required field " <>> name)
|
||
pInputParser parser value
|
||
}
|
||
where
|
||
-- nullable fields just have an implicit default value of `null`
|
||
nullableDefault = case typeNullability (pType parser) of
|
||
Nullable -> Just (GraphQLValue VNull)
|
||
_ -> Nothing
|
||
|
||
-- | Creates a parser for a nullable field with no default value. If the field
|
||
-- is omitted, the provided parser /will not be called/. This allows a field to
|
||
-- distinguish an omitted field from a field supplied with @null@ (which is
|
||
-- permitted by the GraphQL specification); see Note [The value of omitted
|
||
-- fields] for more details.
|
||
--
|
||
-- If you want a field with a default value of @null@, use 'field' with a
|
||
-- nullable value parser, instead.
|
||
fieldOptional ::
|
||
(MonadParse m, 'Input <: k) =>
|
||
Name ->
|
||
Maybe Description ->
|
||
Parser origin k m a ->
|
||
InputFieldsParser origin m (Maybe a)
|
||
fieldOptional name description parser =
|
||
InputFieldsParser
|
||
{ ifDefinitions =
|
||
[ Definition name description Nothing $
|
||
InputFieldInfo (nullableType $ pType parser) Nothing
|
||
],
|
||
ifParser =
|
||
M.lookup name
|
||
>>> withKey (Key (K.fromText (unName name)))
|
||
. traverse (pInputParser parser <=< peelVariable expectedType)
|
||
}
|
||
where
|
||
expectedType = toGraphQLType $ nullableType $ pType parser
|
||
|
||
-- | Creates a parser for an input field with the given default value. The
|
||
-- resulting field will always be optional, even if the underlying parser
|
||
-- rejects `null` values. The underlying parser is always called.
|
||
fieldWithDefault ::
|
||
(MonadParse m, 'Input <: k) =>
|
||
Name ->
|
||
Maybe Description ->
|
||
-- | default value
|
||
Value Void ->
|
||
Parser origin k m a ->
|
||
InputFieldsParser origin m a
|
||
fieldWithDefault name description defaultValue parser =
|
||
InputFieldsParser
|
||
{ ifDefinitions = [Definition name description Nothing $ InputFieldInfo (pType parser) (Just defaultValue)],
|
||
ifParser =
|
||
M.lookup name
|
||
>>> withKey (Key (K.fromText (unName name))) . \case
|
||
Just value -> peelVariableWith True expectedType value >>= pInputParser parser
|
||
Nothing -> pInputParser parser $ GraphQLValue $ literal defaultValue
|
||
}
|
||
where
|
||
expectedType = toGraphQLType $ pType parser
|
||
|
||
-- -----------------------------------------------------------------------------
|
||
-- combinators
|
||
|
||
enum ::
|
||
MonadParse m =>
|
||
Name ->
|
||
Maybe Description ->
|
||
NonEmpty (Definition origin EnumValueInfo, a) ->
|
||
Parser origin 'Both m a
|
||
enum name description values =
|
||
Parser
|
||
{ pType = schemaType,
|
||
pParser =
|
||
peelVariable (toGraphQLType schemaType) >=> \case
|
||
JSONValue (A.String stringValue)
|
||
| Just enumValue <- mkName stringValue -> validate enumValue
|
||
GraphQLValue (VEnum (EnumValue enumValue)) -> validate enumValue
|
||
other -> typeMismatch name "an enum value" other
|
||
}
|
||
where
|
||
schemaType = TNamed NonNullable $ Definition name description Nothing $ TIEnum (fst <$> values)
|
||
valuesMap = M.fromList $ over (traverse . _1) dName $ toList values
|
||
validate value =
|
||
onNothing (M.lookup value valuesMap) $
|
||
parseError $
|
||
"expected one of the values "
|
||
<> englishList "or" (toTxt . dName . fst <$> values)
|
||
<> " for type "
|
||
<> name <<> ", but found " <>> value
|
||
|
||
-- -----------------------------------------------------------------------------
|
||
-- helpers
|
||
|
||
-- TODO: if we had an optional "strict" mode, we could (and should!) enforce
|
||
-- that `fieldName` isn't empty, which sadly can't be done at the type level.
|
||
-- This would prevent the creation of an object with no fields, which is against
|
||
-- the spec.
|
||
object ::
|
||
MonadParse m =>
|
||
Name ->
|
||
Maybe Description ->
|
||
InputFieldsParser origin m a ->
|
||
Parser origin 'Input m a
|
||
object name description parser =
|
||
Parser
|
||
{ pType = schemaType,
|
||
pParser =
|
||
peelVariable (toGraphQLType schemaType) >=> \case
|
||
GraphQLValue (VObject fields) -> parseFields $ GraphQLValue <$> fields
|
||
JSONValue (A.Object fields) -> do
|
||
translatedFields <-
|
||
M.fromList <$> for (KM.toList fields) \(K.toText -> key, val) -> do
|
||
name' <-
|
||
mkName key
|
||
`onNothing` parseError
|
||
("variable value contains object with key " <> key <<> ", which is not a legal GraphQL name")
|
||
pure (name', JSONValue val)
|
||
parseFields translatedFields
|
||
other -> typeMismatch name "an object" other
|
||
}
|
||
where
|
||
schemaType =
|
||
TNamed NonNullable $
|
||
Definition name description Nothing $
|
||
TIInputObject (InputObjectInfo (ifDefinitions parser))
|
||
fieldNames = S.fromList (dName <$> ifDefinitions parser)
|
||
parseFields fields = do
|
||
-- check for extraneous fields here, since the InputFieldsParser just
|
||
-- handles parsing the fields it cares about
|
||
for_ (M.keys fields) \fieldName ->
|
||
unless (fieldName `S.member` fieldNames) $
|
||
withKey (Key (K.fromText (unName fieldName))) $
|
||
parseError $ "field " <> dquote fieldName <> " not found in type: " <> squote name
|
||
ifParser parser fields
|
||
|
||
list :: forall origin k m a. (MonadParse m, 'Input <: k) => Parser origin k m a -> Parser origin k m [a]
|
||
list parser =
|
||
gcastWith
|
||
(inputParserInput @k)
|
||
Parser
|
||
{ pType = schemaType,
|
||
pParser =
|
||
peelVariable (toGraphQLType schemaType) >=> \case
|
||
GraphQLValue (VList values) -> for (zip [0 ..] values) \(index, value) ->
|
||
withKey (Index index) $ pParser parser $ GraphQLValue value
|
||
JSONValue (A.Array values) -> for (zip [0 ..] $ toList values) \(index, value) ->
|
||
withKey (Index index) $ pParser parser $ JSONValue value
|
||
-- List Input Coercion
|
||
--
|
||
-- According to section 3.11 of the GraphQL spec: iff the value
|
||
-- passed as an input to a list type is not a list and not the
|
||
-- null value, then the result of input coercion is a list of
|
||
-- size one, where the single item value is the result of input
|
||
-- coercion for the list's item type on the provided value.
|
||
--
|
||
-- We need to explicitly test for VNull here, otherwise we could
|
||
-- be returning `[null]` if the parser accepts a null value,
|
||
-- which would contradict the spec.
|
||
GraphQLValue VNull -> parseError "expected a list, but found null"
|
||
JSONValue A.Null -> parseError "expected a list, but found null"
|
||
other -> fmap pure $ withKey (Index 0) $ pParser parser other
|
||
}
|
||
where
|
||
schemaType = TList NonNullable $ pType parser
|