NEW: Disable schema introspection in non-dev

[unclecheese]

Disables schema introspection outside of dev. Tests to follow.

[madmatt]

Thanks @unclecheese, from my very uneducated eye this looks good to me! :)

[madmatt]

I merged your suggested change @michalkleiner, this keeps it consistent with the rest. I'm keen to get this merged in, however I just noticed there's a conflict in the README.md file now and realise that there's already a way to remove introspection, it's just done in code and for all environments (though you can use code to amend that).

See https://github.com/silverstripe/silverstripe-graphql#schema-introspection.

Does that invalidate the need for this PR perhaps?

[edited] Actually, on second thought I think that existing feature is designed to stop the dynamic endpoint that shows the introspection dynamically. The point is to make it static as a file in assets/ instead, so this is still required.

[madmatt]

@unclecheese Do you mind if I get this conflict fixed and remove the draft status of this PR, building on top of your work? I've got a few different clients interested in seeing something like this merged in.

Thanks!

[unclecheese]

Yeah, go for it!

[GuySartorelli]

@madmatt are you still keen to fix the conflict on this PR?

[GuySartorelli]

Given that this is still in draft, has conflicts, and is for v3 which we are no longer actively developing, I'm going to close this.
If someone is really passionate about getting this merged in, feel free to open a new PR with no merge conflicts.