[GitHub] nifi issue #1431: Merge pull request #1 from apache/master

2017-01-18 Thread ktseytlin
Github user ktseytlin commented on the issue: https://github.com/apache/nifi/pull/1431 please disregard! --- 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

[GitHub] nifi pull request #1431: Merge pull request #1 from apache/master

2017-01-18 Thread ktseytlin
Github user ktseytlin closed the pull request at: https://github.com/apache/nifi/pull/1431 --- 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

[GitHub] nifi pull request #1431: Merge pull request #1 from apache/master

2017-01-18 Thread ktseytlin
GitHub user ktseytlin opened a pull request: https://github.com/apache/nifi/pull/1431 Merge pull request #1 from apache/master Bringing in Nifi master into local Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the

[GitHub] nifi issue #1112: NIFI-2799: AWS Credentials for Assume Role Need Proxy

2016-10-12 Thread ktseytlin
Github user ktseytlin commented on the issue: https://github.com/apache/nifi/pull/1112 @jvwing I tested with the updated logic on my end and it worked, so I pushed that out in the most recent commit. Does all look good to you now? Any other improvements we can make? --- If your

[GitHub] nifi pull request #1112: NIFI-2799: AWS Credentials for Assume Role Need Pro...

2016-10-11 Thread ktseytlin
Github user ktseytlin commented on a diff in the pull request: https://github.com/apache/nifi/pull/1112#discussion_r82799369 --- Diff: nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/credentials/provider/factory/strategies

[GitHub] nifi pull request #1112: NIFI-2799: AWS Credentials for Assume Role Need Pro...

2016-10-07 Thread ktseytlin
Github user ktseytlin commented on a diff in the pull request: https://github.com/apache/nifi/pull/1112#discussion_r82453054 --- Diff: nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/credentials/provider/factory/strategies

[GitHub] nifi pull request #1112: NIFI-2799: AWS Credentials for Assume Role Need Pro...

2016-10-07 Thread ktseytlin
Github user ktseytlin commented on a diff in the pull request: https://github.com/apache/nifi/pull/1112#discussion_r82451576 --- Diff: nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/credentials/provider/factory/strategies

[GitHub] nifi pull request #1112: NIFI-2799: AWS Credentials for Assume Role Need Pro...

2016-10-07 Thread ktseytlin
Github user ktseytlin commented on a diff in the pull request: https://github.com/apache/nifi/pull/1112#discussion_r82450516 --- Diff: nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/credentials/provider/factory/strategies

[GitHub] nifi pull request #1112: NIFI-2799: AWS Credentials for Assume Role Need Pro...

2016-10-07 Thread ktseytlin
Github user ktseytlin commented on a diff in the pull request: https://github.com/apache/nifi/pull/1112#discussion_r82446283 --- Diff: nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/credentials/provider/factory/strategies

[GitHub] nifi issue #1112: NIFI-2799: AWS Credentials for Assume Role Need Proxy

2016-10-07 Thread ktseytlin
Github user ktseytlin commented on the issue: https://github.com/apache/nifi/pull/1112 This is obviously a pretty awful drawing I threw together... but it shows what AWS is like in an enterprise. For every single service I ever want to execute, I need to go through a proxy. Hence why

[GitHub] nifi pull request #1112: NIFI-2799: AWS Credentials for Assume Role Need Pro...

2016-10-07 Thread ktseytlin
Github user ktseytlin commented on a diff in the pull request: https://github.com/apache/nifi/pull/1112#discussion_r82389180 --- Diff: nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/credentials/provider/factory

[GitHub] nifi pull request #1112: NIFI-2799: AWS Credentials for Assume Role Need Pro...

2016-10-07 Thread ktseytlin
Github user ktseytlin commented on a diff in the pull request: https://github.com/apache/nifi/pull/1112#discussion_r82389187 --- Diff: nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/credentials/provider/factory

[GitHub] nifi pull request #1112: NIFI-2799: AWS Credentials for Assume Role Need Pro...

2016-10-07 Thread ktseytlin
Github user ktseytlin commented on a diff in the pull request: https://github.com/apache/nifi/pull/1112#discussion_r82388965 --- Diff: nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/credentials/provider/factory

[GitHub] nifi pull request #1112: NIFI-2799: AWS Credentials for Assume Role Need Pro...

2016-10-06 Thread ktseytlin
GitHub user ktseytlin opened a pull request: https://github.com/apache/nifi/pull/1112 NIFI-2799: AWS Credentials for Assume Role Need Proxy Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure