markap14 commented on pull request #5324: URL: https://github.com/apache/nifi/pull/5324#issuecomment-1039315005
I think I would be more inclined to accept this PR than to introduce a separate ValidateXMLAttribute processor, as well. If there's concern about adding the character set property, that can be easily addressed by adding an "XML Source" property - either "FlowFile Content" or "FlowFile Attribute" and then make the attribute name dependent, as well as making the "character set" property dependent. That way, we require the Character Set if reading FlowFile content and it's not an option if using an attribute. -- 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]
