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
      7 kB
      Hari Shreedharan
    2. FLUME-2200.patch
      5 kB
      Hari Shreedharan

      Issue Links

        Activity

        Hari Shreedharan created issue -
        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.
        Hari Shreedharan made changes -
        Field Original Value New Value
        Attachment FLUME-2200.patch [ 12605799 ]
        Hari Shreedharan made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Assignee Hari Shreedharan [ hshreedharan ]
        Hide
        Hari Shreedharan added a comment -

        Updated docs

        Show
        Hari Shreedharan added a comment - Updated docs
        Hari Shreedharan made changes -
        Attachment FLUME-2200.patch [ 12605801 ]
        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.
        Hari Shreedharan made changes -
        Remote Link This issue links to "Review (Web Link)" [ 12900 ]
        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!
        Mike Percy made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Fix Version/s v1.5.0 [ 12324642 ]
        Resolution Fixed [ 1 ]
        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
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        9h 10m 1 Hari Shreedharan 29/Sep/13 15:52
        Patch Available Patch Available Resolved Resolved
        9d 5h 10m 1 Mike Percy 08/Oct/13 21:02

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development