[HUE-6055] Need http_client for navigator

Review Request #9770 - Created March 9, 2017 and submitted

Prakash Ranade
enricoberti, jennykim, johan, krish, ranade, romain, subrata, weixia
commit e2ad2ba8abac9d42c26010510fd343794a993b02
Author: Prakash Ranade <ranade@cloudera.com>
Date:   Thu Mar 9 19:08:37 2017 -0800

    [HUE-6055] Need http_client for navigator
    Since navigator api communicate on http and it needs to clear the cookies. In secure cluster clearing cookies will cause service interruption. Seperated navigator's http_client.
    Tested on:

:000000 100644 0000000... c5eb1c1... A	desktop/core/src/desktop/lib/rest/unsecure_http_client.py
:100644 100644 70383f9... b0e4e0e... M	desktop/libs/metadata/src/metadata/navigator_client.py

  • 0
  • 0
  • 3
  • 0
  • 3
Description From Last Updated
  1. Reviews contains a bunch of log statements and is a huge copy paste. Why not use inheritance?

    1. Romain, looking at the time crunch current approach is safer. I will iterate further in 5.12 timeframe.

  1. Some comments to add to explain better the why

  2. HTTP Client that should not be used with Kerberos in order to keep the pool separate from the kerberized one.

    This allow to have a pool of HTTP client with Kerberos authentication and another one without, in case the components are not all secure or all unsecure.

  3. Navigator does not support Kerberos authentication while other components usually requires it

  2. UnsecureHttpClient (unsecure is one workd)

Review request changed

Status: Closed (submitted)