[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-216661583 I'm away from my keyboard right now but I have always used the `maven:shade` plugin. Andy LoPresto alopre...@hortonworks.com

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-216644453 I added a new CLI utility that just hashes passwords using Bcrypt. For the moment, I left the rest of the code alone. I'm still struggling with how to best package and di

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-216594397 My recommendation would be a command line utility (could be a shell script wrapping a JAR) which is invoked without arguments, provides a secure prompt for entering a r

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-216431685 What would you recommend for this pull request? No utility? A simpler hashing utility? --- If your project is set up for it, you can reply to this email and have your re

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r61836807 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/CredentialsStore.java --- @@

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-216417900 I think my earlier comments may have been unclear or ambiguous. I do not believe we need a full command-line interface for modifying the configuration file, as hand-edit

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r61832014 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/CredentialsStore.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r61821120 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/CredentialsCLI.java --- @@ -0

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r61817392 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/CredentialsCLI.java --- @@

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-216353947 I rebased the commits on the master branch to resolve conflicts and use the updated LoginIdentityProvider interface and Administrator's Guide content. I apologize if it co

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-212002570 I have been working (slowly) on the suggested improvements for performance and a reference CLI for basic admin operations. I haven't figured out an elegant way of packagin

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user joewitt commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-211716593 @jvwing @alopresto Have you had a chance to re-engage on this? It seems like a reasonable easy-path option for folks just wanting to use some simple/local username and pa

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r57017952 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r56741436 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r56734268 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-195651563 @jvwing James, you've obviously put effort and thought into this and I didn't mean to deride it. I am working to get the Kerberos authentication completed for the releas

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on the pull request: https://github.com/apache/nifi/pull/267#issuecomment-195596621 @alopresto, thanks for kick-starting the discussion with your feedback, these are great topics. I address some of the concerns you mention below, and ask some questions ba

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55886592 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55884048 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55872279 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55870940 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55868135 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user jvwing commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55863700 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/conf/login-identity-providers.xml --- @@ -89,4 +89,28 @@

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55774945 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55774678 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55773674 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55773583 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55773241 --- Diff: nifi-nar-bundles/nifi-iaa-providers-bundle/nifi-file-identity-provider/src/main/java/org/apache/nifi/authentication/file/FileIdentityProvider.java ---

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

Github user alopresto commented on a diff in the pull request: https://github.com/apache/nifi/pull/267#discussion_r55772695 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/conf/login-identity-providers.xml --- @@ -89,4 +89,28 @@

[GitHub] nifi pull request: NIFI-1614 File Identity Provider implementation

GitHub user jvwing opened a pull request: https://github.com/apache/nifi/pull/267 NIFI-1614 File Identity Provider implementation This implementation of IdentityProvider uses a local XML file to store usernames with bcrypt-hashed passwords. The XML file mechanism is similar to tha