Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/37
Nice work on this refactoring, seems like a good approach.
Noticed one minor thing when I was testing (and didn't really know what I
was doing), I happened to POST to the new token/identity-provider endpoint, but
I still had my URL form-encoding set so it was missing the Basic header, and I
got a NPE:
```
Caused by:
org.apache.nifi.registry.security.authentication.exception.IdentityAccessException:
Unable to validate the supplied credentials. Please contact the system
administrator.
at
org.apache.nifi.registry.security.ldap.LdapIdentityProvider.authenticate(LdapIdentityProvider.java:274)
~[nifi-registry-framework-0.0.1-SNAPSHOT.jar:0.0.1-SNAPSHOT]
at
org.apache.nifi.registry.web.api.AccessResource.createAccessTokenUsingIdentityProviderCredentials(AccessResource.java:220)
~[classes/:na]
... 89 common frames omitted
Caused by: java.lang.NullPointerException: null
at
org.apache.nifi.registry.security.ldap.LdapIdentityProvider.authenticate(LdapIdentityProvider.java:238)
~[nifi-registry-framework-0.0.1-SNAPSHOT.jar:0.0.1-SNAPSHOT]
... 90 common frames omitted
```
I'm guessing the call to extractCredentials returned null, but then not
sure whats supposed to happen after that.
---
