Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: v1.4.0
    • Fix Version/s: None
    • 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
        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
        Hide
        Hari Shreedharan added a comment -

        Committed, rev: 41f1e8afbbc4854c516e1ffd0baf9d0de73d60af. Thanks Ashish!

        Show
        Hari Shreedharan added a comment - Committed, rev: 41f1e8afbbc4854c516e1ffd0baf9d0de73d60af. Thanks Ashish!
        Hide
        Hari Shreedharan added a comment -

        +1. Looks good to go.

        Show
        Hari Shreedharan added a comment - +1. Looks good to go.
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development