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

Permissions cache includes dynamic user variables #9933

Closed
3 tasks done
rijkvanzanten opened this issue Nov 19, 2021 · 0 comments · Fixed by #9957
Closed
3 tasks done

Permissions cache includes dynamic user variables #9933

rijkvanzanten opened this issue Nov 19, 2021 · 0 comments · Fixed by #9957
Assignees
Labels

Comments

@rijkvanzanten
Copy link
Member

Preflight Checklist

Describe the Bug

When using dynamic user variables in permissions, the fetched dynamic variables are included in the permissions cache, meaning that you have to flush the permissions cache by changing the permissions in order to pull in the latest dynamic user variables. The dynamic variables should be fetched dynamically, and should be cached in the user-data cache instead (so it's purged on content change, rather than permissions change).

To Reproduce

Use dynamic variables in permissions, change the data that's used in the permissions, note outdated permissions.

Errors Shown

No response

What version of Directus are you using?

9.0.1

What version of Node.js are you using?

17

What database are you using?

Postgres 13

What browser are you using?

Safari

What operating system are you using?

macOS Big Sur

How are you deploying Directus?

Locally

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants