Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: v1.4.0
    • Fix Version/s: v1.5.0
    • Component/s: None
    • Labels:
      None

      Description

      We should add at least 2 more tests for HTTPS:

      1. Negative test where non-https client fails to connect to https source
      2. Test that events are available on the channel after being written to https

      1. FLUME-2142-0.patch
        4 kB
        Ashish Paliwal

        Activity

        Hari Shreedharan created issue -
        Ashish Paliwal made changes -
        Field Original Value New Value
        Assignee Ashish Paliwal [ paliwalashish ]
        Ashish Paliwal made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Ashish Paliwal made changes -
        Attachment FLUME-2142-0.patch [ 12595814 ]
        Hide
        Ashish Paliwal added a comment -

        Patch available

        Negative test added where non-https client fails to connect to https source
        Test that events are available on the channel after being written to https. Added msg number in header for message sent and checked upon receipt.

        Show
        Ashish Paliwal added a comment - Patch available Negative test added where non-https client fails to connect to https source Test that events are available on the channel after being written to https. Added msg number in header for message sent and checked upon receipt.
        Ashish Paliwal made changes -
        Status In Progress [ 3 ] Patch Available [ 10002 ]
        Hide
        Hari Shreedharan added a comment -

        +1. Looks good to go.

        Show
        Hari Shreedharan added a comment - +1. Looks good to go.
        Hide
        Hari Shreedharan added a comment -

        Committed, rev: 41f1e8afbbc4854c516e1ffd0baf9d0de73d60af. Thanks Ashish!

        Show
        Hari Shreedharan added a comment - Committed, rev: 41f1e8afbbc4854c516e1ffd0baf9d0de73d60af. Thanks Ashish!
        Hari Shreedharan made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Hudson added a comment -

        SUCCESS: Integrated in flume-trunk #484 (See https://builds.apache.org/job/flume-trunk/484/)
        FLUME-2142. HTTPS tests for http source (hshreedharan: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=41f1e8afbbc4854c516e1ffd0baf9d0de73d60af)

        • flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java
        Show
        Hudson added a comment - SUCCESS: Integrated in flume-trunk #484 (See https://builds.apache.org/job/flume-trunk/484/ ) FLUME-2142 . HTTPS tests for http source (hshreedharan: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=41f1e8afbbc4854c516e1ffd0baf9d0de73d60af ) flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java
        Hari Shreedharan made changes -
        Affects Version/s v1.3.1 [ 12323879 ]
        Affects Version/s v1.4.0 [ 12323372 ]
        Fix Version/s v1.5.0 [ 12324642 ]
        Hari Shreedharan made changes -
        Affects Version/s v1.4.0 [ 12323372 ]
        Affects Version/s v1.3.1 [ 12323879 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        2d 11h 22m 1 Ashish Paliwal 04/Aug/13 14:09
        In Progress In Progress Patch Available Patch Available
        2h 34m 1 Ashish Paliwal 04/Aug/13 16:43
        Patch Available Patch Available Resolved Resolved
        1d 5h 31m 1 Hari Shreedharan 05/Aug/13 22:15

          People

          • Assignee:
            Ashish Paliwal
            Reporter:
            Hari Shreedharan
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development