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.

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

Force scope on default filters collection #7511

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

deivid-rodriguez
Copy link
Member

In-repo clone of #6923.

When we use default filters, or filter of an existing relationship without providing a collection, we enforce the collection to be scope
It is useful when you are using authorization scope to define what resources your current user can access, even in filter values.

Fixes #6883.

@javierjulio javierjulio force-pushed the 6883-Scope-default-filter-collections branch from ea831c1 to c3f8cb2 Compare March 13, 2023 00:05
@javierjulio
Copy link
Member

The tests are failing. Likely need some updates since this was last looked at.

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.

Filters values aren't using authorization scopes
3 participants