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

Issue 6913043: state/api: use TLS

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 3 months ago by rog
Modified:
11 years, 3 months ago
Reviewers:
mp+138754
Visibility:
Public.

Description

state/api: use TLS https://code.launchpad.net/~rogpeppe/juju-core/178-api-server/+merge/138754 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : state/api: use TLS #

Patch Set 3 : state/api: use TLS #

Unified diffs Side-by-side diffs Delta from patch set Stats (+58 lines, -10 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M state/api/api_test.go View 2 chunks +3 lines, -3 lines 0 comments Download
M state/api/client.go View 1 chunk +26 lines, -1 line 0 comments Download
M state/api/serve.go View 2 chunks +22 lines, -2 lines 0 comments Download
M testing/cert.go View 2 chunks +4 lines, -3 lines 0 comments Download
M testing/mgo.go View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
rog
Please take a look.
11 years, 3 months ago (2012-12-07 15:27:25 UTC) #1
niemeyer
LGTM
11 years, 3 months ago (2012-12-07 17:57:15 UTC) #2
rog
11 years, 3 months ago (2012-12-07 18:23:04 UTC) #3
*** Submitted:

state/api: use TLS

R=niemeyer
CC=
https://codereview.appspot.com/6913043
Sign in to reply to this message.

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