silverstripe / silverstripe-graphql

Serves Silverstripe data as GraphQL representations
BSD 3-Clause "New" or "Revised" License
52 stars 61 forks source link

ENH Rebuild schema if file missing #530

Closed emteknetnz closed 1 year ago

emteknetnz commented 1 year ago

Issue https://github.com/silverstripe/silverstripe-graphql/issues/500

A better version of https://github.com/silverstripe/silverstripe-graphql/pull/521, which was reverted due to CVE-2023-28104

This version includes a configurable rate-limit that's defaulted to 15 minutes

GuySartorelli commented 1 year ago

behat tests are failing repeatedly

emteknetnz commented 1 year ago

It's an existing failure - https://github.com/silverstripe/silverstripe-graphql/actions/runs/4527398776/jobs/7973214759

Note: After approval I added a little bit to filter out Exception so that they match a particular message. This isn't as good as using a new Exception type, but I didn't want to change the API via changing the Exception thrown, and the message is very unlikely to ever change - https://github.com/silverstripe/silverstripe-graphql/compare/f796b4fd49fbbe6d1b2062d239842a9eea668e6d..0b876c8e2c50f7d69c1eb4805edb5e1de49e5888