[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/250 --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user knusbaum commented on the pull request: https://github.com/apache/storm/pull/250#issuecomment-193481184 @RaghavendraNandagopal Closing this. We still like the functionality, so free to reopen if you have time to address the comments. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/250#discussion_r41551583 --- Diff: storm-core/src/jvm/backtype/storm/Config.java --- @@ -108,7 +108,15 @@ */ public static final String STORM_MESSAGING_NETTY_AUTHENTICATION = "storm.messaging.netty.authentication"; public static final Object STORM_MESSAGING_NETTY_AUTHENTICATION_SCHEMA = Boolean.class; - + +/** + * Netty based messaging: A quality of protection (qop) value is one of "auth" - authentication only, + * "auth-int" - authentication plus integrity protection, "auth-conf" - authentication plus integrity and confidentiality protection. + * The default qop is "auth". + */ +public static final String STORM_MESSAGING_NETTY_PROTECTION = "storm.messaging.netty.protection"; +public static final Object STORM_MESSAGING_NETTY_PROTECTION_SCHEMA = String.class; --- End diff -- * Is this config intended to be a cluster-wide setting or could it be different for each topology? * It would be nice to validate the setting is one of the accepted values beyond being a string. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/250#issuecomment-146651595 @RaghavendraNandagopal , It has been a while, so just checking: Are we still looking to merge this code in? --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/250#discussion_r41551714 --- Diff: storm-core/src/jvm/backtype/storm/messaging/netty/Client.java --- @@ -66,8 +67,14 @@ private AtomicLong flushCheckTimer; private int flushCheckInterval; private ScheduledExecutorService scheduler; + +private CountDownLatch authenticatedSignal = null; + +public CountDownLatch getAuthenticatedSignal() { + return authenticatedSignal; + } -@SuppressWarnings("rawtypes") + @SuppressWarnings("rawtypes") --- End diff -- It seems the existing Java source uses an indentation of four spaces, but the new source uses eight. It would be better I think to keep it the same as the existing source. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/250#issuecomment-76214103 @RaghavendraNandagopal pinging for an upmerge to master. 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/250#issuecomment-69367006 @RaghavendraNandagopal any update on this? Don't want to lose this great work. If you are too busy I can try and upmerge it myself. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user RaghavendraNandagopal commented on the pull request: https://github.com/apache/storm/pull/250#issuecomment-69374819 Hi Bobby, Sorry for getting it late. I will definitely complete it within 3-4 days from now. Hopefully by Tuesday I will send a new pull request. Thanks, Raghavendra Nandagopal --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user RaghavendraNandagopal commented on the pull request: https://github.com/apache/storm/pull/250#issuecomment-63523530 Hi Bobby, Sorry for the delay. I will work on the comments and close it ASAP. Thanks, Raghavendra Nandagopal --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/250#issuecomment-63524216 Not a problem it is great work, I would like to see it in. Also note that security has been merged into master so if you could update your pull request to master that would be great. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] storm pull request: STORM-430: Allow netty SASL to support encrypt...
Github user RaghavendraNandagopal commented on the pull request: https://github.com/apache/storm/pull/250#issuecomment-63528271 Sure, I will work on master branch. Thanks, Raghavendra Nandagopal --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---