Merge lp:~frankban/charms/precise/juju-gui/bug-1095281-restore-tls into lp:~juju-gui/charms/precise/juju-gui/trunk
Proposed by
Francesco Banconi
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 20 | ||||
Proposed branch: | lp:~frankban/charms/precise/juju-gui/bug-1095281-restore-tls | ||||
Merge into: | lp:~juju-gui/charms/precise/juju-gui/trunk | ||||
Diff against target: |
95 lines (+12/-20) 5 files modified
README.md (+0/-2) config/config.js.template (+1/-1) config/nginx.conf.template (+8/-12) hooks/start (+1/-2) tests/deploy.test (+2/-3) |
||||
To merge this branch: | bzr merge lp:~frankban/charms/precise/juju-gui/bug-1095281-restore-tls | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju GUI Hackers | Pending | ||
Review via email: mp+141646@code.launchpad.net |
Description of the change
Restore TLS in the charm
Now the charm is served using SSL again.
To post a comment you must log in.
Reviewers: mp+141646_ code.launchpad. net,
Message:
Please take a look.
Description:
Restore TLS in the charm
Now the charm is served using SSL again.
https:/ /code.launchpad .net/~frankban/ charms/ precise/ juju-gui/ bug-1095281- restore- tls/+merge/ 141646
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/7026046/
Affected files: config. js.template nginx.conf. template
M README.md
A [revision details]
M config/
M config/
M hooks/start
M tests/deploy.test
Index: README.md
machine: 1
open-ports:
public- address: ec2-204- 236-250- 8.compute- 1.amazonaws. com
=== modified file 'README.md'
--- README.md 2012-12-20 18:02:44 +0000
+++ README.md 2013-01-02 13:31:03 +0000
@@ -58,9 +58,7 @@
- 80/tcp
- <!--- Uncomment when TLS connections are re-enabled.
- 443/tcp
- -->
- 8080/tcp
Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision:
<email address hidden>
+New revision:
<email address hidden>
Index: config/ config. js.template config. js.template' config. js.template 2012-11-30 16:36:47 +0000 config. js.template 2013-01-02 13:31:03 +0000 jujucharms. com/', /jujucharms. com/', (address) s:%(port) s/ws'
=== modified file 'config/
--- config/
+++ config/
@@ -5,6 +5,6 @@
container: '#main',
viewContainer: '#main',
transitions: false,
- charm_store_url: 'http://
+ charm_store_url: 'https:/
socket_url: 'ws://%
};
Index: config/ nginx.conf. template nginx.conf. template' nginx.conf. template 2012-12-21 15:22:50 +0000 nginx.conf. template 2013-01-02 13:31:03 +0000 /$host$ request_ uri; /$host$ request_ uri;
=== modified file 'config/
--- config/
+++ config/
@@ -1,20 +1,16 @@
-# Uncomment to switch back to TLS connections.
-# server {
-# listen 80;
-# server_name _;
-# return 301 https:/
-# }
+server {
+ listen 80;
+ server_name _;
+ return 301 https:/
+}
server { path)s/ server. pem; path)s/ server. key; path)s/ server. pem; path)s/ server. key;
- # Uncomment to switch back to TLS connections.
- # listen 443 default_server ssl;
- listen 80; # Delete this line when TLS connections are re-enabled.
+ listen 443 default_server ssl;
server_name _;
root %(server_root)s;
index index.html;
- # Uncomment to switch back to TLS connections.
- # ssl_certificate %(ssl_cert_
- # ssl_certificate_key %(ssl_cert_
+ ssl_certificate %(ssl_cert_
+ ssl_certificate_key %(ssl_cert_
# Serve static assets.
location ^~ /juju-ui/ {
Index: hooks/start
=== modified file 'hooks/start'
--- hooks/start 2012-12-21 16:32:53 +0000
+++ hooks/start 2013-01-02 13:31:03 +0000
@@ -21,8 +21,7 @@
log('Exposing services.')
# Open the Juju GUI web server HTTP and HTTPS ports.
open_port(80)
- # Uncomment to switch back to TLS connections.
- # open_port(443)
+ open_port(443)
# Open the Juju websocket server port.
o...