Patch to add SSL support to HTTP Source, Flume-2109
Review Request #12244 - Created July 3, 2013 and discarded
Patch for Flume-2109, to add HTTPS support to HTTP Source
Unit Test added to verify SSL messages being sent to the Source.
Posted (July 10, 2013, 1:28 a.m.)
Ashish, This looks pretty good. Right now it looks like both SSL and non-SSL connectors will be enabled if SSL is enabled. I don't think we should have both enabled - if SSL is enabled only secure connections should be possible. That is how even the Avro Source works right now.
Review request changed
Updated (July 10, 2013, 7:36 a.m.)
- changed from pending to discarded
Discarding this review request to fix review comments