HUE-8585 [useradmin] Bubbling up errors for Add Sync Ldap Users

Review Request #13374 - Created Sept. 20, 2018 and updated

Ying Chen
hue
master
HUE-8585
hue
enricoberti, jgauthier, johan, ranade, romain, weixia
Author: Ying Chen <yingchen@cloudera.com>
Date:   Thu Sep 20 17:23:24 2018 -0700

    HUE-8585 [useradmin] Bubbling up errors for Add Sync Ldap Users

:100644 100644 a8cf6e82f0... 389d937908... M    apps/useradmin/src/useradmin/templates/add_ldap_users.mako
:100644 100644 3cf33b96cf... c414e1b134... M    apps/useradmin/src/useradmin/views.py
Test LDAP Bind Exception
  • 1
  • 0
  • 0
  • 0
  • 1
Description From Last Updated
Why not bubbling up the exception message and adding a generic? ' please check server log for detail.' Romain Rigaux
  1. 
      
  2. apps/useradmin/src/useradmin/views.py (Diff revision 1)
     
     

    Why not bubbling up the exception message and adding a generic?
    ' please check server log for detail.'

    1. The detail message is like "Failed to bind to LDAP server as user adminUser@domain.example.com", so this is to not send sensitive information like LDAP config to UI.
  3. 
      
Loading...