Merge lp:~ce-infrastructure/capomastro/capomastro-apache-1385220 into lp:~ubuntuone-hackers/capomastro/charm
Proposed by
Caio Begotti
Status: | Merged |
---|---|
Approved by: | Daniel Manrique |
Approved revision: | 55 |
Merged at revision: | 52 |
Proposed branch: | lp:~ce-infrastructure/capomastro/capomastro-apache-1385220 |
Merge into: | lp:~ubuntuone-hackers/capomastro/charm |
Diff against target: |
125 lines (+67/-10) 4 files modified
README (+22/-7) hooks/website-relation-changed (+6/-0) hooks/website-relation-joined (+0/-3) templates/http_virtualhost.tmpl (+39/-0) |
To merge this branch: | bzr merge lp:~ce-infrastructure/capomastro/capomastro-apache-1385220 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel Manrique (community) | Approve | ||
Review via email: mp+241059@code.launchpad.net |
Description of the change
This change makes it possible to have Capomastro sitting behind another Apache server, from the charms store, acting like an exposed proxy. This is the first step to get SSO in Capomastro without needing to change its authentication codo right now.
Also, this is another requirement for production by IS: not to expose your own web server, and use a supported one via charming.
SSL support is still pending because we need a review of it all by IS and their advices on that.
To post a comment you must log in.
I couldn't get SSO support on time for this merge request but since it's not a requirement for #1385220 I'll keep investigating how SSO could be attached to this in parallel, this merge request stands on its own anyway.