Change from plain `casbin_rule` to `openedx_authz.casbin_rule`

[mariajgrimaldi]

Description

Currently the table name for the policies is casbin_rule, which misses all the context of where the table is from. I believe we should change the table name from casbin_rule to openedx_authz.casbin_rule for additional context.

[mariajgrimaldi]

Hi folks @BryanttV @bmtcril! What do you think of this?

[bmtcril]

Makes sense to me if it's not a complicated change

[BryanttV]

Yes, I agree with @bmtcril.

[BryanttV]

Hi! I was checking how we could change the table name, and here’s what I found:

The Django ORM Adapter doesn’t allow configuring it, since the table name is hardcoded . So it’s not possible to modify it through any setting.

I can think of two possible options:

• Inherit from the original model and change the table name. However, this would also require overriding the calls to the original model, which are currently made only inside the Adapter. The problem with this approach is that if the internal logic of those methods changes for any reason in the package, our code could easily get out of sync.

• Contribute this change directly to the django-orm-adapter package.

And finally, the last option would be to simply keep the current table name. What do you think? Do you see any other possible approaches?

[bmtcril]

I think that this is low priority compared to the other remaining work and would just leave it as-is for now.