You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by zenfenan <gi...@git.apache.org> on 2018/12/12 14:53:07 UTC

[GitHub] nifi pull request #2968: NIFI-5456: AWS clients now work with private link e...

Github user zenfenan commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2968#discussion_r241044902
  
    --- Diff: nifi-nar-bundles/nifi-aws-bundle/nifi-aws-abstract-processors/src/main/java/org/apache/nifi/processors/aws/AbstractAWSProcessor.java ---
    @@ -286,7 +286,7 @@ protected void initializeRegionAndEndpoint(ProcessContext context) {
                 final String urlstr = StringUtils.trimToEmpty(context.getProperty(ENDPOINT_OVERRIDE).evaluateAttributeExpressions().getValue());
                 if (!urlstr.isEmpty()) {
                     getLogger().info("Overriding endpoint with {}", new Object[]{urlstr});
    -                this.client.setEndpoint(urlstr);
    +                this.client.setEndpoint(urlstr, this.client.getServiceName(), this.region.getName());
    --- End diff --
    
    Thanks @amcdonaldccri for pointing this out. I'll work on fixing it.


---