Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

count resolver does not verify that req.passed is not false after calling next. #5

Open
robere2 opened this issue Mar 3, 2023 · 0 comments
Open

count resolver does not verify that req.passed is not false after calling next. #5

robere2 opened this issue Mar 3, 2023 · 0 comments
Labels
Complexity: LOW This issue is trivial and is expected to take less than an hour to complete. good first issue This issue is expected to be easy to complete for a new-comer to the relevant module Module: API This issue pertains to the apps/api module Priority: HIGH This issue is halting multiple issues, significantly hindering users, or is a minor security issue. Type: Bug Something isn't working

Comments

@robere2
Copy link
Member

robere2 commented Mar 3, 2023

Currently, resolvers must verify that req.passed is not set to false after calling next. The count rule handler does not do this.
@robere2 robere2 added Type: Bug Something isn't working Priority: MED This issue isn't critical or security-related but is holding back other issues or efficiency. good first issue This issue is expected to be easy to complete for a new-comer to the relevant module labels Mar 3, 2023
@robere2 robere2 transferred this issue from rpitv/glimpse-api Jan 1, 2024
@robere2 robere2 added Priority: HIGH This issue is halting multiple issues, significantly hindering users, or is a minor security issue. Module: API This issue pertains to the apps/api module and removed Priority: MED This issue isn't critical or security-related but is holding back other issues or efficiency. labels Jan 1, 2024
robere2 added a commit that referenced this issue Jan 7, 2024
@robere2
Merge pull request #5 from rpitv/add-graphql
5e0b096 
Added GraphQL client w/ IDE introspection
robere2 added a commit that referenced this issue Jan 7, 2024
@robere2
Merge pull request #5 from rpitv/add-graphql
8914f91 
Added GraphQL client w/ IDE introspection
@robere2 robere2 added the Complexity: LOW This issue is trivial and is expected to take less than an hour to complete. label Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Complexity: LOW This issue is trivial and is expected to take less than an hour to complete. good first issue This issue is expected to be easy to complete for a new-comer to the relevant module Module: API This issue pertains to the apps/api module Priority: HIGH This issue is halting multiple issues, significantly hindering users, or is a minor security issue. Type: Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@robere2