- Issue created by @chfoidl
- Merge request !7Implement access policy to add access ui route permission for CLI requests #3504339 → (Open) created by chfoidl
- 🇧🇪Belgium dieterholvoet Brussels
Thanks for the contribution! Wouldn't it be simpler to just add the CLI check to
DisableUiAccessCheckinstead of addingCliAccessPolicy, or is there a specific reason you went down that route? - 🇦🇹Austria chfoidl Salzburg
That's a good point. The reason I did not implement it directly was because I was not 100% certain if the AccessCheck is never cached, because if it is, the access check would return false results.
If you can assure that the AccessCheck is never cached, the logic can be moved to the AccessCheck.
- 🇧🇪Belgium dieterholvoet Brussels
It might be cached, but you can add the cache context you provided to the access result.