manifold-systems / manifold

Manifold is a Java compiler plugin, its features include Metaprogramming, Properties, Extension Methods, Operator Overloading, Templates, a Preprocessor, and more.
http://manifold.systems/
Apache License 2.0
2.43k stars 125 forks source link

manifold-graphql: larger graphql models will not compile due to graphql-java default 15000 token limit #536

Closed rsmckinney closed 11 months ago

rsmckinney commented 11 months ago

graphql-java parser added a default token limit of 15000 (to address DoS), one manifold-sql customer has graphql models that exceed this limit, which causes graphql compilation failure.

Since manifold parses graphql exclusively at compile time, this limit can be ignored, thus we can make it sufficiently large as to never reach it in practice. Other runtime specific limits may need addressing as well. See ParserOptions.newParserOptions().

rsmckinney commented 11 months ago

Fix available with release 2023.1.33