You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@knox.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2019/11/05 03:28:00 UTC

[jira] [Work logged] (KNOX-1842) Upgrade httpclient to 4.5.10

     [ https://issues.apache.org/jira/browse/KNOX-1842?focusedWorklogId=338476&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-338476 ]

ASF GitHub Bot logged work on KNOX-1842:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 05/Nov/19 03:27
            Start Date: 05/Nov/19 03:27
    Worklog Time Spent: 10m 
      Work Description: risdenk commented on pull request #176: KNOX-1842 - Upgrade httpclient to 4.5.10
URL: https://github.com/apache/knox/pull/176
 
 
   ## What changes were proposed in this pull request?
   
   HttpClient 4.5.7 broke url normalization. Knox
   didn't have any tests for this case and so we
   had to revert after the fact. HttpClient 4.5.8
   fixed a lot of the url normalization and some
   libraries decided to turn url normalization off.
   
   This commit does the following:
   * Adds a test for %2F - KNOX-1005
       * This test passes under HttpClient 4.5.6 and 4.5.8+
       * It breaks as expected under HttpClient 4.5.7
   * Adds an explicit config enabling url normalization
       * Ensures that we are in control of url normalization
       * Adds a test for this configuration as well
   * Test with both HttpClient normalization enabled and disabled
       * `rest-assured` doesn't expose `RequestConfig` to disable
   url normalization
       * Shows how to use HttpClient in `GatewayBasicFuncTest`
   
   All the url safe characters like %2F are fixed by HTTPCLIENT-1968.
   
   The case of `/abc///def` is normalized to `/abc/def` the same
   way that Knox does internally with `getPathInfo` and Java URI.
   
   ## How was this patch tested?
   
   * Tested against multiple versions of HttpClient (4.5.6, 4.5.7, and 4.5.10)
   * `mvn -T.5C clean verify -Ppackage,release -Dshellcheck`
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Issue Time Tracking
-------------------

            Worklog Id:     (was: 338476)
    Remaining Estimate: 0h
            Time Spent: 10m

> Upgrade httpclient to 4.5.10
> ----------------------------
>
>                 Key: KNOX-1842
>                 URL: https://issues.apache.org/jira/browse/KNOX-1842
>             Project: Apache Knox
>          Issue Type: Sub-task
>            Reporter: Kevin Risden
>            Assignee: Kevin Risden
>            Priority: Minor
>             Fix For: 1.4.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Upgrade httpclient 4.5.6 to 4.5.10



--
This message was sent by Atlassian Jira
(v8.3.4#803005)