Skip to content
This repository has been archived by the owner on Jul 25, 2020. It is now read-only.

Check downloaded archive with GPG. #12

Closed
wants to merge 4 commits into from
Closed

Check downloaded archive with GPG. #12

wants to merge 4 commits into from

Conversation

ypid
Copy link
Collaborator

@ypid ypid commented Mar 23, 2015

@silvio
Copy link
Contributor

silvio commented May 6, 2015

Hi I have reviewed your patch.
Can you following thing change?

  • please rebase
  • download the https://www.owncloud.org/owncloud.asc file, and don't include it completly. It is possible that the gpg keys alter in the future.
  • Introduce a version variable and use them for downloading of asc-file and archive.
  • Don't use a makefile. Do all things in the Dockerfile

ypid added 3 commits May 6, 2015 19:12
* This did not work for 7.0.4 so this commit also upgrades to Owncloud
  8.0.2. See #10 for dissuasion about v8.0.
* Just run `make verify-gpg-public-keys` and check if the (re)downloaded
  files have changed in git (they should not change unless owncloud
  changed there keys!).
@ypid
Copy link
Collaborator Author

ypid commented May 6, 2015

Not including the signing key from ownCloud or at least specifying the fingerprint does destroy the whole purpose of checking the signature in the first place. What would happen when in the Dockerfile all three files are downloaded over https and someone hacks there website? Right, the attacker would create a new public/private key, sign the archive with it and upload the public key … When we remove the public key from this repo, there is no benefit in security …

I tried to verify the authenticity of the public key as good as I can and the commit which added the file is GPG signed with my private key.

Please reconsider …

@silvio
Copy link
Contributor

silvio commented May 6, 2015

Yeah okay. You are right. The altering of gpg keys should not happen frequently.

@ypid ypid mentioned this pull request May 6, 2015
@ypid
Copy link
Collaborator Author

ypid commented May 24, 2015

Not yet merged?

ypid added a commit to ypid/docker-ejabberd that referenced this pull request May 30, 2015
* GPG verification was added in 7650b36
  but unfortunately has been broken after this commit.
* See jchaney/owncloud#12
@ypid ypid closed this May 30, 2015
silvio pushed a commit to silvio/docker-owncloud that referenced this pull request Jun 1, 2015
@ypid ypid deleted the gpg-verify-owncloud branch August 6, 2015 21:08
ypid added a commit to ypid/docker-torbrowser that referenced this pull request Sep 13, 2015
ypid added a commit to ypid/ansible-nodejs that referenced this pull request Sep 23, 2015
* Testing against Debian Wheezy.
* Made ready for new DebOps documentation format.
* Don’t download the PGP key from the same source as the packages. See
  jchaney/owncloud#12 (comment)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants