Uploaded image for project: 'Flume'
  1. Flume
  2. FLUME-2142

HTTPS tests for http source

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

            Activity

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

            +1. Looks good to go.

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

            Committed, rev: 41f1e8afbbc4854c516e1ffd0baf9d0de73d60af. Thanks Ashish!

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