Flume
  1. Flume
  2. FLUME-2142

HTTPS tests for http source

        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. Text File
            FLUME-2142-0.patch
            4 kB
            Ashish Paliwal

            Activity

            Descending order - Click to sort in ascending order
            Hide
            Permalink
            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
            Permalink
            Hari Shreedharan added a comment -

            Committed, rev: 41f1e8afbbc4854c516e1ffd0baf9d0de73d60af. Thanks Ashish!

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

            +1. Looks good to go.

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