Skip to content

Use graphql Guard to remove auth checks from resolvers - #210

Open
tpoliaw wants to merge 1 commit into
mainfrom
auth_guards
Open

tpoliaw wants to merge 1 commit into
mainfrom
auth_guards

Conversation

@tpoliaw

@tpoliaw tpoliaw commented Aug 22, 2025

Copy link
Copy Markdown
Collaborator

Using the attributes allows the auth logic to be separated from the
logic in the resolver endpoints and makes it easier to re-use the checks
for other future endpoints if they're added.

@codecov

codecov Bot commented Oct 30, 2025 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 76.19048% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.55%. Comparing base (b8b38d9) to head (9da86f0).

Files with missing lines Patch % Lines
src/graphql/auth.rs 76.19% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #210      +/-   ##
==========================================
+ Coverage   79.50%   79.55%   +0.05%     
==========================================
  Files          13       13              
  Lines        2132     2128       -4     
==========================================
- Hits         1695     1693       -2     
+ Misses        437      435       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fajinyuan

Copy link
Copy Markdown
Contributor

could you resolve the conflicts please. I read them but don't quiet understand to be able to resolve them. Thank you

Using the attributes allows the auth logic to be separated from the
logic in the resolver endpoints and makes it easier to re-use the checks
for other future endpoints if they're added.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants