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

Review Request #13599 - Created Feb. 6, 2019 and submitted

Ying Chen
hue
master
HUE-8585
hue
jgauthier, johan, romain, weixia
commit e2f43bace27e83a16fe8cf3b82b5ec6f8b882ec2 (HEAD -> master)
Author: Ying Chen <yingchen@cloudera.com>
Date:   Wed Feb 6 16:41:30 2019 -0800

    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 0672b283e5... 54e38e7f53... M    apps/useradmin/src/useradmin/templates/list_users.mako
:100644 100644 3cf33b96cf... 61367b379c... M    apps/useradmin/src/useradmin/views.py
:100644 100644 640fec65fc... 8fbe9dc59d... M    desktop/core/src/desktop/auth/forms.py


  • 0
  • 0
  • 2
  • 0
  • 2
Description From Last Updated
  1. BTW: Is there a quick test ?

    1. Yes, Chris suggested an LDAP config to trigger this exceptions, and I was able to reproduce it.
  2. apps/useradmin/src/useradmin/views.py (Diff revision 1)
     
     

    smart_str(e) did not work?

    Seems weird to convert to utf8 and inject a string with str(e)?

  3. apps/useradmin/src/useradmin/views.py (Diff revision 1)
     
     

    Missing %s

  4. 
      
  1. 
      
  2. 
      
Review request changed

Status: Closed (submitted)

Loading...