Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(927)

Issue 14524043: Preliminary metadata signing (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 6 months ago by wallyworld
Modified:
10 years, 6 months ago
Reviewers:
fwereade, mp+189755
Visibility:
Public.

Description

Preliminary metadata signing Add an Encode funtion which can be used to sign simplestreams metadata. Use the new Encode function in tests which set up signed metadata and verify that image and tools information can be read. This functionality was until now not formally tested due to lack of ability to sign arbitary metadata in juju. https://code.launchpad.net/~wallyworld/juju-core/signed-simplestreams-metadata-tests/+merge/189755 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+451 lines, -93 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
A environs/imagemetadata/export_test.go View 1 chunk +11 lines, -0 lines 0 comments Download
M environs/imagemetadata/simplestreams.go View 2 chunks +60 lines, -0 lines 0 comments Download
M environs/imagemetadata/simplestreams_test.go View 3 chunks +134 lines, -0 lines 0 comments Download
M environs/simplestreams/decode.go View 2 chunks +2 lines, -61 lines 0 comments Download
A environs/simplestreams/encode.go View 1 chunk +50 lines, -0 lines 1 comment Download
M environs/simplestreams/export_test.go View 1 chunk +0 lines, -7 lines 0 comments Download
M environs/simplestreams/simplestreams.go View 9 chunks +20 lines, -11 lines 0 comments Download
M environs/simplestreams/simplestreams_test.go View 2 chunks +4 lines, -14 lines 0 comments Download
M environs/simplestreams/testing/testing.go View 1 chunk +41 lines, -0 lines 0 comments Download
M environs/tools/export_test.go View 1 chunk +7 lines, -0 lines 0 comments Download
M environs/tools/simplestreams.go View 2 chunks +10 lines, -0 lines 1 comment Download
M environs/tools/simplestreams_test.go View 3 chunks +110 lines, -0 lines 0 comments Download

Messages

Total messages: 3
wallyworld
Please take a look.
10 years, 6 months ago (2013-10-08 04:38:11 UTC) #1
fwereade
Confused about the empty public key and what will actually happen if we run this ...
10 years, 6 months ago (2013-10-09 08:15:45 UTC) #2
fwereade
10 years, 6 months ago (2013-10-09 08:18:06 UTC) #3
On 2013/10/09 08:15:45, fwereade wrote:
> Confused about the empty public key and what will actually happen if we run
this
> live. Otherwise looking solid.
> 
>
https://codereview.appspot.com/14524043/diff/1/environs/simplestreams/encode.go
> File environs/simplestreams/encode.go (right):
> 
>
https://codereview.appspot.com/14524043/diff/1/environs/simplestreams/encode....
> environs/simplestreams/encode.go:45: err = plaintext.Close()
> Should this have been deferred earlier?
> 
> https://codereview.appspot.com/14524043/diff/1/environs/tools/simplestreams.go
> File environs/tools/simplestreams.go (right):
> 
>
https://codereview.appspot.com/14524043/diff/1/environs/tools/simplestreams.g...
> environs/tools/simplestreams.go:41: var simplestreamsToolsPublicKey =
> `-----BEGIN PGP PUBLIC KEY BLOCK-----
> What happens here in practice, live?

As discussed live, this is moot because tools data isn't signed yet anyway. LGTM
for trunk and 1.16
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b