[
https://issues.apache.org/jira/browse/STORM-348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14078531#comment-14078531
]
ASF GitHub Bot commented on STORM-348:
--------------------------------------
Github user revans2 commented on the pull request:
https://github.com/apache/incubator-storm/pull/202#issuecomment-50549920
The code looks good. The wrap/unwrap we can save for a later time. I had
some very minor merge conflicts and
storm-core/test/clj/backtype/storm/messaging/netty_unit_test.clj is failing for
me. It looks like it needs to have STORM-MESSAGING-NETTY-AUTHENTICATION false
added to all of the configs, as they are unit tests and don't pick up
defaults.yaml
> (Security) Netty SASL Authentication
> ------------------------------------
>
> Key: STORM-348
> URL: https://issues.apache.org/jira/browse/STORM-348
> Project: Apache Storm (Incubating)
> Issue Type: Bug
> Reporter: Robert Joseph Evans
> Assignee: Raghavendra Nandagopal
> Labels: security
> Attachments: Storm-Netty Authentication.pdf
>
>
> Currently The Netty transport does no authentication at all. You can encrypt
> the tuples being sent, but that is a huge performance hit for many cases that
> do not need it. We should support simple SASL authentication when Netty
> first connects to an external process. We probably want to use something
> similar to what we do for ZK, and generate a random secret for each topology.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
