Github user kevdoran commented on the issue:

    https://github.com/apache/nifi-registry/pull/37
  
    Good catch! I added a null check that makes the LdapIdentityProvider 
implementation handle this gracefully. The next PR (#41) makes this even a bit 
more robust by adding a null check in the /access/token/identity-provider as 
well.
    
    Thanks for the feedback, let me know if you see anything else to change 
prior to merging in. There are a couple deprecated classes (exception types) 
that I plan to address in a follow up PR as there were already a ton of classes 
touched here and I didn't want to introduce the noise of a bunch of classes 
with one-line changes to exception types in this PR.


---

Reply via email to