-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[BUG] Access Control lower cases resources intermittently #6004
Comments
Hey @JoeGaffney thanks for the issue. We are open to contributions for this one. |
Hey @BatuhanW |
Hey @JayBhensdadia assigned to you. Please make sure to thoroughly read and follow our contribution guide. PRs that doesn't follow our contribution guide, will be closed. |
sure |
This issue is related with our sider components ( |
ah that makes sense with the pattern I was seeing ( some cases changes and others not) I did have a resource defined with only a show and not a list in the resources that was camel and was not affected. So would not appear in the side panel and thus probably not go through the canAccess first from the sider |
Describe the bug
Sometimes resource names can get changed to lower case even though they are Camel cased.
Will then appear as auditlogs in the access control Can method
Steps To Reproduce
Create a camelCase resouce.
Log the resource name in the accessControl.
export const accessControlProvider = {{
can: async ({{ resource, action, params }}) => {{
console.log(resource)
console.log("Resource Name:", params?.resource?.name);
...
}},
}};
Expected behavior
Not to change the casing
Packages
Additional Context
Was mentioned on discord to put a ticket in @aliemir said he has a solution.
Potentially somehow related to kbar?
The text was updated successfully, but these errors were encountered: