-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fix(GraphQL): Nested Auth Rules not working properly. (#7915) (#8084) #8571
Conversation
|
@anurags92 could you add more explanation to the commit for what is going on? |
@mangalaman93 I have context on this, iirc, this was related to an issue I reported via DM on Discuss. Let me see if I can find you and add you to the conversation. |
If I added both of your correct users @anurags92 and @mangalaman93 then you both should now have access to: https://discuss.dgraph.io/t/security-bug-auth-security-rule-not-honored-correctly/13372/11?u=amaster507 |
That helps, thank you so much. |
What is this exactly? That link is dead. is this related to https://discuss.dgraph.io/t/bug-auth-rules-of-parent-not-respected-when-child-with-hasinverse-is-added/12955 ? J |
Improves nested auth rule implementation in graphql.
(cherry picked from commit e7a1931)
Co-authored-by: minhaj-shakeel minhaj@dgraph.io
(cherry picked from commit 26845c4)