Having custom validation over scalars

[mxamin]

I need to pass some validation check when defining Arguments for Mutation. I know I can check it inside the mutate function but I need this check before that (similar to what we have in marshmallow.validate module: Range and Length).

I just want to know if there is any future plan to implement it inside the graphene or not.

[KaySackey]

You can create a custom scalar that does validation. It’s what I do anyways, when accepting input strings to validate that user data doesn’t contain HTML.

[mxamin]

But the scalars won't accept inputs, suppose I want to define a new scalar which accepts strings with specific lengths, in that case, I want to pass the length value to my custom scalar but as far as I found out, scalars won't accept input parameters and you have to hardcode the length.
Correct me if I'm wrong :)

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[KaySackey]

Not sure what you are doing in particular, but Scalars can accept kwargs. You just have to access them via self.kwargs

[elchiapp]

Sure, you can pass kwargs but where exactly would the validation logic run and be called? parse_value, serialize? Also there's no access to the supertype so what if I wanted to show a hierarchy/path to the wrong value (e.g. items[0].email is invalid)?

[jkimbo]

@chpmrc do you need to validate before the mutate function? (just trying to understand the use case better)

[elchiapp]

Yes, similarly to how Django validates input in the forms, before calling the various create, update, etc. I threw together a decorator for pre mutation validation for graphene, I'll see if I can open a PR once it's polished.

[jkimbo]

I was going to suggest using a decorator! I think that is the easiest approach to add validation.

I'd be interested in seeing a PR if you get it working.

[elchiapp]

@jkimbo so I came up with this PoC and I was hoping you could take a look to maybe integrate it in Graphene (after some refinements): https://github.com/chpmrc/graphene-validator . Let me know what you think! (I put the repo together quickly but we're already using it in production)

[elchiapp]

@jkimbo did you have a chance to look at the repo I linked above? Cheers!

[vancouverwill]

Sure, you can pass kwargs but where exactly would the validation logic run and be called? parse_value, serialize? Also there's no access to the supertype so what if I wanted to show a hierarchy/path to the wrong value (e.g. items[0].email is invalid)?

@chpmrc I got the validation we needed working by using a custom Scalar and adding the validation on parse_value() as that does get called by the Graphene library before input is passed to either query or mutations. Adding an example here as the docs are super sparse and might of help to you or someone else in the future

class Email(String):

    @staticmethod
    def parse_value(value: Text) -> Text:
        if is_valid_email(value):
            return value
        raise GraphQLError(
                f"Invalid Graph ID {value}, should be hexadecimal format 0x1f1f"
            )

def is_valid_email(email):
    # do some regex or other validation

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.