tailcallhq / tailcall

High Performance GraphQL Runtime
https://tailcall.run
Apache License 2.0
1.3k stars 254 forks source link

feat(jit): plan ahead auth requirements when required #3139

Closed karatakis closed 3 days ago

karatakis commented 4 days ago

Summary:
We plan the authentication requirements for the query plan, and we can execute them before we make any calls to remove services.

Issue Reference(s):
Fixes https://github.com/tailcallhq/tailcall/issues/3125

Build & Testing:

Checklist:

codecov[bot] commented 4 days ago

Codecov Report

Attention: Patch coverage is 85.33333% with 11 lines in your changes missing coverage. Please review.

Project coverage is 86.58%. Comparing base (847d1da) to head (216810f). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/core/jit/transform/auth_planer.rs 81.13% 10 Missing :warning:
src/core/jit/exec_const.rs 92.85% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #3139 +/- ## ========================================== - Coverage 86.60% 86.58% -0.02% ========================================== Files 276 277 +1 Lines 27097 27164 +67 ========================================== + Hits 23467 23520 +53 - Misses 3630 3644 +14 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features: