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

fix: casbin migration #1245

Merged
merged 1 commit into from
Jan 2, 2025
Merged

fix: casbin migration #1245

merged 1 commit into from
Jan 2, 2025

Conversation

adityathebe
Copy link
Member

@adityathebe adityathebe commented Jan 2, 2025

## fixes: 

failed to initialize rbac: 
error creating rbac enforcer: invalid policy rule size: expected 6, got 5, 
rule: [everyone database.kratos * deny true]

setting v5 as empty string didn't fix this. 
Casbin probably still sees the row as 5 columns, so setting it as "na".

Policies formed from permissions table have a valid UUID for v5 and mission-control uses uuid.Validate() to filter them out.

failed to initialize rbac:
error creating rbac enforcer: invalid policy rule size: expected 6, got 5,
rule: [everyone database.kratos * deny true]

setting v5 as empty string didn't fix this.
Casbin probably still sees the row as 5 columns, so setting it as "na".

Policies formed from permissions table have a valid UUID for v5 and mission-control uses uuid.Validate() to filter them out.
@adityathebe adityathebe force-pushed the feat/casbin-migration branch from 7c41cca to ec288ed Compare January 2, 2025 11:37
@adityathebe adityathebe requested a review from moshloop January 2, 2025 11:48
@moshloop moshloop merged commit 5b3a60f into main Jan 2, 2025
7 checks passed
@moshloop moshloop deleted the feat/casbin-migration branch January 2, 2025 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants