graphql: Log warning about result size before erroring

This makes it possible to see in the logs what was rejected by setting both env variables to the same value
graphprotocol · Oct 20, 2022 · 90d359c · 90d359c
1 parent 34c7c90
commit 90d359c
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/graphql/src/store/prefetch.rs b/graphql/src/store/prefetch.rs
@@ -514,15 +514,15 @@ fn check_result_size<'a>(
     ctx: &'a ExecutionContext<impl Resolver>,
     size: usize,
 ) -> Result<(), QueryExecutionError> {
+    if size > ENV_VARS.graphql.warn_result_size {
+        warn!(ctx.logger, "Large query result"; "size" => size, "query_id" => &ctx.query.query_id);
+    }
     if size > ENV_VARS.graphql.error_result_size {
         return Err(QueryExecutionError::ResultTooBig(
             size,
             ENV_VARS.graphql.error_result_size,
         ));
     }
-    if size > ENV_VARS.graphql.warn_result_size {
-        warn!(ctx.logger, "Large query result"; "size" => size, "query_id" => &ctx.query.query_id);
-    }
     Ok(())
 }