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.

Sign up for GitHub

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

Jump to bottom

Access Control: Clear user's permission cache after resource creation #59307

Merged
merged 1 commit into from Nov 24, 2022
Merged

Access Control: Clear user's permission cache after resource creation #59307

merged 1 commit into from Nov 24, 2022

Conversation

grafanabot
Copy link
Contributor

Backport a8bae3f from #59101

@IevaVasiljeva @grafanabot
Access Control: Clear user's permission cache after resource creation (…
777347f 
…#59101)

* refresh user's permission cache after resource creation

* clear the cache instead of reloading the permissions

* don't error if can't clear cache

* fix tests

* fix tests again

(cherry picked from commit a8bae3f)
@grafanabot grafanabot requested review from a team as code owners November 24, 2022 14:41
@grafanabot grafanabot requested review from mgyongyosi and Jguer and removed request for a team November 24, 2022 14:41
@grafanabot grafanabot added this to the 9.3.0 milestone Nov 24, 2022
@grafanabot grafanabot added backport A backport PR no-changelog Skip including change in changelog/release notes type/bug area/backend labels Nov 24, 2022
@IevaVasiljeva IevaVasiljeva enabled auto-merge (squash) November 24, 2022 14:45
@grafanabot
Copy link
Contributor Author

Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/43298

@IevaVasiljeva IevaVasiljeva merged commit 2e36890 into v9.3.x Nov 24, 2022
@IevaVasiljeva IevaVasiljeva deleted the backport-59101-to-v9.3.x branch November 24, 2022 14:52
@IevaVasiljeva IevaVasiljeva added add to changelog and removed no-changelog Skip including change in changelog/release notes labels Nov 24, 2022
@IevaVasiljeva IevaVasiljeva changed the title [v9.3.x] Access Control: Clear user's permission cache after resource creation Access Control: Clear user's permission cache after resource creation Nov 24, 2022
GuYounes pushed a commit to paul-wurth/BIXpert that referenced this pull request Feb 8, 2023
@grafanabot @IevaVasiljeva @GuaYounesPW
[v9.3.x] Access Control: Clear user's permission cache after resource…
7ce7ffc 
… creation (grafana#59307)

Access Control: Clear user's permission cache after resource creation (grafana#59101)

* refresh user's permission cache after resource creation

* clear the cache instead of reloading the permissions

* don't error if can't clear cache

* fix tests

* fix tests again

(cherry picked from commit a8bae3f)

Co-authored-by: Ieva <ieva.vasiljeva@grafana.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IevaVasiljeva IevaVasiljeva IevaVasiljeva approved these changes

@mgyongyosi mgyongyosi Awaiting requested review from mgyongyosi mgyongyosi is a code owner automatically assigned from grafana/grafana-authnz-team

@Jguer Jguer Awaiting requested review from Jguer Jguer is a code owner automatically assigned from grafana/grafana-authnz-team

Assignees
No one assigned
Labels
add to changelog area/backend backport A backport PR enterprise-failed type/bug
Projects
None yet
Milestone
9.3.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@grafanabot @IevaVasiljeva