turcsanyip commented on a change in pull request #3601: NIFI-6468: Adding AWS
S3 'requester pays' to Fetch and List processors.
URL: https://github.com/apache/nifi/pull/3601#discussion_r307530419
##########
File path:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-abstract-processors/src/main/java/org/apache/nifi/processors/aws/s3/AbstractS3Processor.java
##########
@@ -126,6 +126,15 @@
new AllowableValue("S3SignerType", "Signature v2"))
.defaultValue("Default Signature")
.build();
+ public static final PropertyDescriptor REQUESTER_PAYS = new
PropertyDescriptor.Builder()
+ .name("requester-pays")
+ .displayName("Requester Pays")
+ .description("If set to 'True', the 'x-amz-request-payer' header
will be set to 'requester', indicating that you agree to pay S3 bucket request "
+ + " charges. Note that this is not supported when
listing object versions.")
+ .required(true)
Review comment:
As listing versions and "requester pays" do not work together, it would be
useful to add a custom validator to ListS3 which checks that these two
properties cannot be true at the same time.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
