nifi-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (NIFI-786) Add other supporting options for configuring credentials for AWS processors
Date Sat, 21 May 2016 01:01:12 GMT


ASF GitHub Bot commented on NIFI-786:

Github user jskora commented on the pull request:
    I'm starting to review and test this.

> Add other supporting options for configuring credentials for AWS processors
> ---------------------------------------------------------------------------
>                 Key: NIFI-786
>                 URL:
>             Project: Apache NiFi
>          Issue Type: Improvement
>    Affects Versions: 0.3.0
>            Reporter: Michael Kobit
>            Assignee: James Wing
>            Priority: Minor
>             Fix For: 0.7.0
> I was looking at and looked at how the
AWS processors credentials are currently configured. As a NFM you have a few options with
the properties right now:
> 1) set basic, static credentials
> 2) set a credentials properties filepath
> 3) set neither, use anonymous credentials
> I think it would be better if each AWS could rely on a ControllerService that returns
`AWSCredentialsProvider` (instead of  `AWSCredentials`) that gives all of the possible implementations
that could be used, rather than relying on a static credentials. *Provider implementations
can be refreshed and can also  other more complicated implementations, but already have built
in support for the Static and Properties file that are provided by NiFi today.
> My thinking is that the controller service would be something like
> public interface AwsCredentialsProviderService extends ControllerService {
> 	AWSCredentialsProvider getCredentialsProvider();
> }
> and you could have `StaticAwsCredentialsProviderService`, `PropertiesFileAwsCredentialsProviderService`,
and `AnonymousAwsCredentialsProviderService` to provide the functionality that is supported
right now. Additional credential providers could be added later, as there a bunch more AWS
provided versions that I think could fit in well.

This message was sent by Atlassian JIRA

View raw message