Actions
Issue #7845
closedThe view_set parameter on AccessPolicy is not namespacing plugins
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 86
Quarter:
Description
We need to use another deterninant here because e.g. PackageViewSet will appear in multiple plugins.
In: ContainerDistributionViewSet.view_name()
Out: 'distributions-container/container'
would do the job.
If no plugin is using AccessPolicy by now, we can just switch the lookup here. Otherwise we'd need to be very careful with deprecating it, because a failed lookup will result in "500" errors.
Updated by fao89 about 4 years ago
- Triaged changed from No to Yes
- Sprint set to Sprint 86
Updated by pulpbot about 4 years ago
- Status changed from ASSIGNED to POST
Added by mdellweg about 4 years ago
Updated by mdellweg about 4 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulpcore|7f738e07ce5798734c958168f8809d84245379cc.
Updated by pulpbot almost 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Namespace vieset_name in AccessPolicy
To avoid naming conflicts with similarly named viewsets in plugins, a scoping by the plugin app_id is needed.
fixes #7845 https://pulp.plan.io/issues/7845