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

PK uuid and auth_permission #437

Open
josejachuf opened this issue Jun 16, 2022 · 1 comment
Open

PK uuid and auth_permission #437

josejachuf opened this issue Jun 16, 2022 · 1 comment
Milestone

Comments

@josejachuf
Copy link

Taking into account the UUID type PK support. How is it resolved in auth_permission record_id (integer) field?

Sample of docs:

maintenance = db.Preference(name='maintenance').first()
auth.add_permission(admins, 'write', 'Preference', maintenance.id)
auth.has_permission('write', 'Preference', maintenance.id)

If Preference.id is uuid, this would not work.

Jose

@gi0baro
Copy link
Member

gi0baro commented Jun 17, 2022

@josejachuf you right, I didn't take into account permissions.

This sounds a bit complex, I need to spend some time on this in order to find a backward compatible way of supporting this. Ideally will be in 2.5

@gi0baro gi0baro added this to the 2.5 milestone Jun 17, 2022
@gi0baro gi0baro modified the milestones: 2.5, 2.6 Mar 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants