Github user apiri commented on the pull request:
https://github.com/apache/nifi/pull/290#issuecomment-222586092
Hey @trixpan,
Scoped out your new changes and left a couple comments. Changes look good,
just had a question on whether the SSL Context should be made required and
there is, I believe, the unused message demarcator field. Let me know your
thoughts on those and we can get this incorporated and closed out.
Thanks!
---
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.
---
