Flume
  1. Flume
  2. FLUME-2200

HTTP Source should be able to use "port" parameter if SSL is enabled

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: v1.5.0
    • Component/s: None
    • Labels:
      None
    1. FLUME-2200.patch
      5 kB
      Hari Shreedharan
    2. FLUME-2200.patch
      7 kB
      Hari Shreedharan

      Issue Links

        Activity

        Hide
        Hari Shreedharan added a comment -

        Patch fixing this issue. Tests modified to support this change.

        Show
        Hari Shreedharan added a comment - Patch fixing this issue. Tests modified to support this change.
        Hide
        Hari Shreedharan added a comment -

        Updated docs

        Show
        Hari Shreedharan added a comment - Updated docs
        Hide
        Jeff Lord added a comment -

        +1 I didn't see a review board link though.

        Show
        Jeff Lord added a comment - +1 I didn't see a review board link though.
        Hide
        Hari Shreedharan added a comment -

        Note that the sslPort parameter was never part of a release, so this patch does not break compatibility.

        Show
        Hari Shreedharan added a comment - Note that the sslPort parameter was never part of a release, so this patch does not break compatibility.
        Hide
        Mike Percy added a comment -

        +1

        Show
        Mike Percy added a comment - +1
        Hide
        Mike Percy added a comment -

        Pushed to trunk and flume-1.5 branches. Thanks for the patch Hari!

        Show
        Mike Percy added a comment - Pushed to trunk and flume-1.5 branches. Thanks for the patch Hari!
        Hide
        Hudson added a comment -

        FAILURE: Integrated in flume-trunk #509 (See https://builds.apache.org/job/flume-trunk/509/)
        FLUME-2200. HTTP Source should use "port" param for both SSL & cleartext (mpercy: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=20eed3fdcbee57b84504ec0e1adada46950c4f90)

        • flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSourceConfigurationConstants.java
        • flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java
        • flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java
        • flume-ng-doc/sphinx/FlumeUserGuide.rst
        Show
        Hudson added a comment - FAILURE: Integrated in flume-trunk #509 (See https://builds.apache.org/job/flume-trunk/509/ ) FLUME-2200 . HTTP Source should use "port" param for both SSL & cleartext (mpercy: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=20eed3fdcbee57b84504ec0e1adada46950c4f90 ) flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSourceConfigurationConstants.java flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java flume-ng-doc/sphinx/FlumeUserGuide.rst

          People

          • Assignee:
            Hari Shreedharan
            Reporter:
            Hari Shreedharan
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development