Github user miquillo commented on the issue:
https://github.com/apache/nifi/pull/362
@jvwing Thanks for picking this up!
- I would 100% apply this to the AbstractS3processor. By the time I figured
it myself, I already had implemented it (in the wrong class). Should be easy to
move to the AbstractS3processor, not much code is affected by this PR.
- Definitely make this a combobox / selectable option. It was just a quick
fix to us to make it a true/false flag as we are only using Signature v4.
Indeed makes it futureproof.
- I am completely unaware of the ClientConfiguration::setSignerOverride()
and I do also dislike the system.setproperty(..). If it works the same way;
definitely go for it.
---
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 enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
