RE: documentation feedback

2017-03-15 Thread Brian Cornally
Hi Colin, 

Sorry, just reading this now. Looks like I don't have permissions to commit to 
a new branch to make pull request. I haven't made pull request before so may be 
doing it incorrectly ;)

git push origin doc-security-console-example
remote: Permission to apache/kafka.git denied to briancornally.
fatal: unable to access 'https://github.com/apache/kafka.git/': The requested 
URL returned error: 403

$ git diff origin
diff --git a/docs/security.html b/docs/security.html
index 2e5d492..5118b57 100644
--- a/docs/security.html
+++ b/docs/security.html
@@ -213,8 +213,12 @@
 
 Examples using console-producer and console-consumer:
 
-kafka-console-producer.sh --broker-list localhost:9093 --topic 
test --producer.config client-ssl.properties
-kafka-console-consumer.sh --bootstrap-server localhost:9093 
--topic test --consumer.config client-ssl.properties
+echo ssl | kafka-console-producer.sh --broker-list localhost:9093 
--topic test --producer.config client-ssl.properties
+kafka-console-consumer.sh --bootstrap-server localhost:9093 
--topic test --consumer.config client-ssl.properties
+
+notes:
+- older version of kafka-console-producer.sh requires additional 
option --security-protocol SSL
+- older version of kafka-console-consumer.sh requires additional 
options --security-protocol SSL --new-consumer
 
 
 7.3 Authentication using 
SASL

Best, 
 
Brian

-Original Message-
From: Colin McCabe [mailto:cmcc...@apache.org] 
Sent: Friday, February 17, 2017 11:55 AM
To: dev@kafka.apache.org
Subject: Re: documentation feedback

Hi Brian,

Have you created a pull request for this?

best,
Colin


On Thu, Feb 9, 2017, at 15:21, Brian Cornally wrote:
> @apachekafka suggested addition to
> http://kafka.apache.org/documentation/#security - section 5 - Examples 
> using console-producer and console-consumer:
> 
> echo "ssl" |  
> /usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh
> --broker-list $KAFKABROKERSSSL --topic test2  --producer.config 
> client-ssl.properties --security-protocol SSL
> note: addition of --security-protocol SSL
> 
> /usr/hdp/current/kafka-broker/bin/kafka-console-consumer.sh 
> --zookeeper $KAFKAZKHOSTS --bootstrap-server $KAFKABROKERSSSL --topic 
> test2 --from-beginning --consumer.config client-ssl.properties 
> --security-protocol SSL --new-consumer
> note: addition of --security-protocol SSL --new-consumer Thanks
> 


Re: documentation feedback

2017-02-17 Thread Colin McCabe
Hi Brian,

Have you created a pull request for this?

best,
Colin


On Thu, Feb 9, 2017, at 15:21, Brian Cornally wrote:
> @apachekafka suggested addition to
> http://kafka.apache.org/documentation/#security - section 5 - Examples
> using console-producer and console-consumer:
> 
> echo "ssl" |  /usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh
> --broker-list $KAFKABROKERSSSL --topic test2  --producer.config
> client-ssl.properties --security-protocol SSL
> note: addition of --security-protocol SSL
> 
> /usr/hdp/current/kafka-broker/bin/kafka-console-consumer.sh --zookeeper
> $KAFKAZKHOSTS --bootstrap-server $KAFKABROKERSSSL --topic test2
> --from-beginning --consumer.config client-ssl.properties
> --security-protocol SSL --new-consumer
> note: addition of --security-protocol SSL --new-consumer
> Thanks
>