Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.4.0
    • Fix Version/s: 1.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

        Hide
        paliwalashish 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
        paliwalashish 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.
        Hide
        hshreedharan Hari Shreedharan added a comment -

        +1. Looks good to go.

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

        Committed, rev: 41f1e8afbbc4854c516e1ffd0baf9d0de73d60af. Thanks Ashish!

        Show
        hshreedharan Hari Shreedharan added a comment - Committed, rev: 41f1e8afbbc4854c516e1ffd0baf9d0de73d60af. Thanks Ashish!
        Hide
        hudson 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 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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development