Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Download .zip" fails on charms with terms #355

Open
kwmonroe opened this issue Oct 6, 2016 · 5 comments
Open

"Download .zip" fails on charms with terms #355

kwmonroe opened this issue Oct 6, 2016 · 5 comments
Labels
Milestone

Comments

@kwmonroe
Copy link

kwmonroe commented Oct 6, 2016

Click the download .zip button on a charm with terms, eg:

https://jujucharms.com/u/ibmcharmers/ibm-http/trusty/5

The request will fail. If you try to curl/wget the link, you'll see an error about macaroons:

wget https://api.jujucharms.com/charmstore/v5/~ibmcharmers/trusty/ibm-http-5/archive

This feels like a bug to me. I understand that deployment from the store should be gated on term acceptance, but surely we should be able to download the charm archive. We can already click on all the files and save manually -- why block on downloading them all at once with 'Download .zip'?

@urosj urosj added the bug label Oct 6, 2016
@urosj
Copy link

urosj commented Oct 6, 2016

We need to properly verify terms agreement, which we currently don't.

@johnsca
Copy link

johnsca commented Oct 7, 2016

@urosj I thought our story was that the charm code was considered open source and agreeing to the terms was only required for deploying it? That is, it seems to me that the terms agreement applies to the resource payload rather than the charm code itself.

@frankban frankban added this to the 1.2.0 milestone Mar 3, 2017
@cmars
Copy link

cmars commented Jun 8, 2018

Bump. Ran into this today when trying to download cs:~cmars/mattermost. Actually it was Tim but didn't work for me either. Looks like the web page (https://jujucharms.com/u/cmars/mattermost/16) isn't discharging the third-party caveat on the terms service, even if terms have already been agreed to.

@cmars
Copy link

cmars commented Jun 8, 2018

@frankban
Copy link

This is still an issue, and we need to have a conversation about how to fix this.
The storefront (for anonymous users) does not have any notion of bakery auth, so we maybe should trigger a GUI authentication when clicking links to get resources that require a discharge. But the GUI itself, who could easily provide the required macaroons, fails to initiate the macaroon acquisition process. Therefore I created a bug in the GUI as well: juju/juju-gui#3760

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants