Hive
  1. Hive
  2. HIVE-6592

WebHCat E2E test abort when pointing to https url of webhdfs

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.13.0
    • Fix Version/s: 0.13.0, 0.14.0
    • Component/s: Tests, WebHCat
    • Labels:
      None

      Description

      WebHCat E2E tests when running against a ssl enabled webhdfs url fails.
      NO PRECOMMIT TESTS

      1. HIVE-6592.patch
        0.6 kB
        Deepesh Khandelwal

        Activity

        Deepesh Khandelwal created issue -
        Hide
        Deepesh Khandelwal added a comment -

        The patch appends a "-k" to the curl command when making requests to https urls. The option allows curl to make insecure ssl connections and transfers.

        Show
        Deepesh Khandelwal added a comment - The patch appends a "-k" to the curl command when making requests to https urls. The option allows curl to make insecure ssl connections and transfers.
        Deepesh Khandelwal made changes -
        Field Original Value New Value
        Attachment HIVE-6592.patch [ 12633500 ]
        Deepesh Khandelwal made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Fix Version/s 0.13.0 [ 12324986 ]
        Deepesh Khandelwal made changes -
        Description WebHCat E2E tests when running against a ssl enabled webhdfs url fails.
        WebHCat E2E tests when running against a ssl enabled webhdfs url fails.
        NO PRECOMMIT TESTS
        Hide
        Thejas M Nair added a comment -

        +1

        Show
        Thejas M Nair added a comment - +1
        Hide
        Sushanth Sowmyan added a comment -

        Looks good to me from reading up "man curl" :

               -k/--insecure
                      (SSL) This option explicitly allows curl to perform "insecure" SSL connections and transfers. All SSL connections  are  attempted  to  be
                      made  secure  by  using  the  CA  certificate  bundle  installed by default. This makes all connections considered "insecure" fail unless
                      -k/--insecure is used.
        
                      See this online resource for further details: http://curl.haxx.se/docs/sslcerts.html
        

        +1.

        Show
        Sushanth Sowmyan added a comment - Looks good to me from reading up "man curl" : -k/--insecure (SSL) This option explicitly allows curl to perform "insecure" SSL connections and transfers. All SSL connections are attempted to be made secure by using the CA certificate bundle installed by default. This makes all connections considered "insecure" fail unless -k/--insecure is used. See this online resource for further details: http://curl.haxx.se/docs/sslcerts.html +1.
        Hide
        Sushanth Sowmyan added a comment -

        Committed to trunk, Thanks, Deepesh! Setting the fix version to 0.14.

        Harish Butani, Deepesh would like to get this included in 0.13 as well. I think it makes sense for inclusion, since it's needed to allow our E2E tests to run in a secure environment. Could we backport this?

        Show
        Sushanth Sowmyan added a comment - Committed to trunk, Thanks, Deepesh! Setting the fix version to 0.14. Harish Butani , Deepesh would like to get this included in 0.13 as well. I think it makes sense for inclusion, since it's needed to allow our E2E tests to run in a secure environment. Could we backport this?
        Sushanth Sowmyan made changes -
        Fix Version/s 0.14.0 [ 12326450 ]
        Fix Version/s 0.13.0 [ 12324986 ]
        Sushanth Sowmyan made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Harish Butani added a comment -

        +1 for 0.13

        Show
        Harish Butani added a comment - +1 for 0.13
        Hide
        Sushanth Sowmyan added a comment -

        Thanks Harish and Deepesh, committed to 0.13.

        Show
        Sushanth Sowmyan added a comment - Thanks Harish and Deepesh, committed to 0.13.
        Sushanth Sowmyan made changes -
        Fix Version/s 0.13.0 [ 12324986 ]
        Hide
        Thejas M Nair added a comment -

        This has been fixed in 0.14 release. Please open new jira if you see any issues.

        Show
        Thejas M Nair added a comment - This has been fixed in 0.14 release. Please open new jira if you see any issues.
        Thejas M Nair made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Deepesh Khandelwal
            Reporter:
            Deepesh Khandelwal
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development