hydrazine doesn't check for the access granted

Bug #643610 reported by Vincent Ladeuil
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Hydrazine
Fix Released
Low
Martin Packman

Bug Description

I installed hydrazine in a new environment and made a mistake: I grant *read* access for eveything.
This didn't sound silly at the time...

Then I encounter problem with feed-pqm, some weird traceback (sorry I didn't keep it) when sending a submission
by mail.

But the submission was sent anyway !

Since I was in a hurry I forgot about it for a while until I try making a submission again and *then* I noticed a 401 HTTP error. I suspect it was when feed-pqm tried to post the 'sent by mail' comment to the mp.

So I remove the .cache/hydrazine dir and started again, granted write access and viola, it works.

Related branches

Revision history for this message
Martin Pool (mbp) wrote : Re: [Bug 643610] [NEW] hydrazine doesn't check for the access granted

launchpadlib now lets the app specify what access it wants. Perhaps
hydrazine should just ask for all access?

Revision history for this message
Vincent Ladeuil (vila) wrote :

At least feed-pqm should, since it intends to post on the user behalf.

Revision history for this message
Max Bowsher (maxb) wrote :

Should it ask for access to private things?

Revision history for this message
Martin Pool (mbp) wrote : Re: [Bug 643610] Re: hydrazine doesn't check for the access granted

On 21 September 2010 19:03, Max Bowsher <email address hidden> wrote:
> Should it ask for access to private things?

If you mean "should it get access to private bugs" then I think yes,
that's clearly useful.

If you mean "should it check with the user before requesting access to
private bugs" then I think it depends on whether the user would ever
want to refuse access to private bugs. Since hydrazine is intended to
run directly under the control of the user and they have the source, I
don't really see why they would want to refuse. So let's keep it
simple for now.

--
Martin

Martin Packman (gz)
Changed in hydrazine:
assignee: nobody → Martin Packman (gz)
importance: Undecided → Low
status: New → In Progress
Martin Packman (gz)
Changed in hydrazine:
status: In Progress → 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.