unembargo-package can't move package-diffs out of restricted librarian

Bug #270992 reported by Celso Providelo
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Undecided
Celso Providelo

Bug Description

The current code fails to move package diffs from restricted librarian (to the public instance) because it has no permission to override IPackageDiff.diff_content, because it's proxied by the security adapter.

The fact that the object is proxied in zopeless more is intriguing, but a quick solution would be to use removeSecurityProxy() for now and land it ASAP.

Tags: lp-soyuz
Celso Providelo (cprov)
Changed in soyuz:
assignee: nobody → cprov
milestone: none → 2.1.9
status: New → In Progress
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

All zopeless code still uses security proxies. It's just that the security policy is different (all attributes available through the proxy are automatically allowed).

So here I think it's more defining these attributes as writable in the permission system.

Revision history for this message
Celso Providelo (cprov) wrote :

RF 7025

Changed in soyuz:
status: In Progress → Fix Released
Revision history for this message
Celso Providelo (cprov) wrote :

Oh, this revision wasn't released to soyuz production because of the r7021 issue.

Changed in soyuz:
status: Fix Released → Fix Committed
Celso Providelo (cprov)
Changed in soyuz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.