markobean commented on pull request #5324: URL: https://github.com/apache/nifi/pull/5324#issuecomment-1036460776
I am a -1 for an additional `ValidateXMLAttribute` processor. Such a processor adds to processor bloat, and it does not solve any of the current resistance outlined in this PR. In fact, it creates more problems than it solves. Another negative to such a named processor is the increased visibility into the fact that XML in attributes is supported. This will only exacerbate the concerns of uninformed users voiced by @markap14. Another nudge to the +1 direction of this PR is the fact that the processor handles attributes is not obvious unless/until a user looks at the configuration options and documentation. And, to be explicit, the processor documentation clearly has a warning for use of large attribute values by making use of the `@SystemResourceConsideration`. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
