Ashish, Please attach the patch to the jira. I will commit it.
Hari Thanks, Hari On Thursday, August 1, 2013 at 7:05 PM, Hari Shreedharan wrote: > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12803/ > > > > > Ship it! > As discussed, please file a follow up jira to add a new test. please update > the log message to INFO and attach the patch to the jira. > > - Hari Shreedharan > On July 22nd, 2013, 7:56 a.m. UTC, Ashish Paliwal wrote: > Review request for Flume. > By Ashish Paliwal. > > Updated July 22, 2013, 7:56 a.m. > > Bugs: FLUME-2109 (https://issues.apache.org/jira/browse/FLUME-2109) > Repository: flume-git > Description > Fix for Flume-2109, to support HTTPS. Incorporates Review comments from Alex > Implementation refined to be as close to Avro Source, like SSL port is not to > be defined, the port value shall be used Documentation updated to reflect the > changes > > > > > Testing > Unit Test case written to test SSL functionality > > > > > Diffs > flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java > (c90f067) > flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSourceConfigurationConstants.java > (f547e0f) > flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java > (8952db3) > flume-ng-doc/sphinx/FlumeUserGuide.rst (63cad21) > > View Diff (https://reviews.apache.org/r/12803/diff/) > > > > > > >
