ppa-dev-tools.ppa tests always asks for auth

Bug #2022363 reported by Dimitri John Ledkov
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
ppa-dev-tools
Fix Released
Undecided
Unassigned

Bug Description

ppa-dev-tools.ppa tests always asks for auth

it seems like the snap has no access to gnome-keyring / $HOME or some such, and doesn't store the token locally, thus each time i invoke `tests` i have do reauth.

Pleas store the token upon use anywhere. Ideally gnome-keyring.

Revision history for this message
Bryce Harrington (bryce) wrote :

Yes, Lucas ran into this same issue with the snap. I think he ended up using the package from the ppa-dev-tools PPA since it won't have this problem.

Thanks for reporting this; I mostly dogfood from git so didn't notice it when testing the snap.

Changed in ppa-dev-tools:
status: New → Confirmed
Revision history for this message
Bryce Harrington (bryce) wrote :

I'm marking this bug as 'bitesize' as it looks like an issue that is easy to fix and suitable for newcomers in Ubuntu development. If you need any help with fixing it, talk to me about it.

tags: added: bitesize
tags: removed: bitesize
tags: added: bitesize
tags: removed: bitesize
Revision history for this message
Bryce Harrington (bryce) wrote :

I tested one of the ubuntu-dev-tools scripts run in an LXC environment and found it has the same problem. So I'm not sure this is specific to snap, and may be something in launchpadlib rather than in the client itself. I don't rule out that there may be a way to work around it by manually doing the Credentials handling that launchpadlib's login_with() should be doing.

Revision history for this message
Bryce Harrington (bryce) wrote :

I've implemented the solution described in comment #4, and this is now available in the 0.5 release.

Essentially, on a properly logged in system you can run `ppa credentials`, which writes out a credentials.oauth file. Copy that to your target system where you want to run ppa-dev-tools, then do `ppa --credentials ./credentials.oauth <command> ...`

There may well be additional polish required to make the snap work more seamlessly, so definitely reopen this if the solution does not feel complete yet.

Changed in ppa-dev-tools:
status: Confirmed → Fix Released
Revision history for this message
Adrien Nader (adrien) wrote :

It would be interesting to be able specify where to store the credential file. Right now it's possible to pass an argument after "ppa credentials" and it will fail with "Error: Invalid ppa name '{args.ppa_name}'". Another way to address this would be to forbid that extra argument.

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.