> On March 5, 2014, 7:43 a.m., John Kinsella wrote: > > core/src/com/cloud/info/ConsoleProxyInfo.java, line 45 > > <https://reviews.apache.org/r/18759/diff/1/?file=510344#file510344line45> > > > > Can you clean up trailing whitespace added in lines like this?
Will do. IDE issues :( > On March 5, 2014, 7:43 a.m., John Kinsella wrote: > > systemvm/conf/consoleproxy.properties, line 19 > > <https://reviews.apache.org/r/18759/diff/1/?file=510349#file510349line19> > > > > Why change the listening port? I don't see what this has to do with > > moving from HTTPS to HTTP... The issue arises in code base, where are the some parts where the port of console proxy listening service is hard coded to 80 / 443 based on HTTP / HTTPS. 8088 existed for legacy reasons - the developer had used it for local testing and it stayed put. - Amogh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18759/#review36212 ----------------------------------------------------------- On March 5, 2014, 1:42 a.m., Amogh Vasekar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18759/ > ----------------------------------------------------------- > > (Updated March 5, 2014, 1:42 a.m.) > > > Review request for cloudstack, Demetrius Tsitrelis and John Kinsella. > > > Repository: cloudstack-git > > > Description > ------- > > Changes to support HTTP mode in Console Proxy, per > http://www.mail-archive.com/dev@cloudstack.apache.org/msg24151.html , as > realhostip may go away > > More details here : > https://cwiki.apache.org/confluence/display/CLOUDSTACK/Realhost+IP+changes > > > Diffs > ----- > > core/src/com/cloud/info/ConsoleProxyInfo.java 3439f3d > > engine/storage/image/src/org/apache/cloudstack/storage/image/TemplateServiceImpl.java > a649bb7 > > plugins/storage/image/default/src/org/apache/cloudstack/storage/datastore/driver/CloudStackImageStoreDriverImpl.java > 52cad3b > server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java 3572127 > server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java > a3f837b > systemvm/conf/consoleproxy.properties bb452f5 > > Diff: https://reviews.apache.org/r/18759/diff/ > > > Testing > ------- > > Tested on local environment by > 1. Using HTTP based console proxy > 2. HTTPS with realhostip domain > 3. HTTPS with custom domain and self-signed cert > 4. Secondary storage template download with custom domain and self-signed cert > > > Thanks, > > Amogh Vasekar > >