fix: pass response to Http::execute() in GraphQL resolver#11878
fix: pass response to Http::execute() in GraphQL resolver#11878
Conversation
Http::execute() now requires a Response parameter as of utopia-php/http 0.34.20. The GraphQL resolver was only passing route and request, causing all GraphQL queries to fail with "Internal server error". Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Greptile SummaryThis PR fixes a one-line regression in the GraphQL resolver where Confidence Score: 5/5Safe to merge — targeted one-line fix with no side effects.
No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "fix: pass response to Http::execute() in..." | Re-trigger Greptile |
🔄 PHP-Retry SummaryFlaky tests detected across commits: Commit
|
| Test | Retries | Total Time | Details |
|---|---|---|---|
UsageTest::testFunctionsStats |
1 | 10.21s | Logs |
UsageTest::testPrepareSitesStats |
1 | 7ms | Logs |
UsageTest::testEmbeddingsTextUsageDoesNotBreakProjectUsage |
1 | 5ms | Logs |
DatabasesStringTypesTest::testCreateTable |
1 | 272.76s | Logs |
TablesDBCustomClientTest::testManyToManyRelationship |
1 | 1.02s | Logs |
TablesDBCustomServerTest::testNotBetween |
1 | 240.62s | Logs |
TablesDBTransactionsCustomClientTest::testBulkUpsertOperations |
1 | 240.40s | Logs |
✨ Benchmark results
⚡ Benchmark Comparison
|
Http::execute() now requires a Response parameter as of utopia-php/http 0.34.20. The GraphQL resolver was only passing route and request, causing all GraphQL queries to fail with "Internal server error".
What does this PR do?
(Provide a description of what this PR does and why it's needed.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)
Related PRs and Issues
Checklist