Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/317#issuecomment-204498317
@mcgilman I agree - we certainly need to address the duplication of
configuration. However, I do believe that is a job for another ticket. I went
ahead and added username/password configs for zookeeper into the
nifi.properties file for now. We can figure out how best to refactor the
configuration in a later ticket before 1.0 is released.
I did push a new commit, which I believe addresses the issue that you laid
out, though, about auth failures. I also found an edge case that wasn't being
handled very well, so I addressed that as well in this latest commit and added
a unit test to verify.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
