[ 
https://issues.apache.org/jira/browse/KAFKA-3396?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15287873#comment-15287873
 ] 

Edoardo Comar commented on KAFKA-3396:
--------------------------------------

Hi [~granthenke] we're still working on it, 
we're working to get the unit tests pass (and added a few others) .

We had some surprises running `SaslSslEndToEndAuthorizationTest`
if we change the consumers from 
```consumers.head.assign(List(tp).asJava)```
to
```consumers.head.subscribe(List(topic).asJava) ```
the code paths are different and even the original tests may not pass unless we 
change
```  this.serverConfig.setProperty(KafkaConfig.MinInSyncReplicasProp, "1")```
to be a `"1"`
which is the case also for the original code. Not sure if we're missing 
something.


> Unauthorized topics are returned to the user
> --------------------------------------------
>
>                 Key: KAFKA-3396
>                 URL: https://issues.apache.org/jira/browse/KAFKA-3396
>             Project: Kafka
>          Issue Type: Bug
>            Reporter: Grant Henke
>
> Kafka's clients and protocol exposes unauthorized topics to the end user. 
> This is often considered a security hole. To some, the topic name is 
> considered sensitive information. Those that do not consider the name 
> sensitive, still consider it more information that allows a user to try and 
> circumvent security.  Instead, if a user does not have access to the topic, 
> the servers should act as if the topic does not exist. 
> To solve this some of the changes could include:
>       - The broker should not return a TOPIC_AUTHORIZATION(29) error for 
> requests (metadata, produce, fetch, etc) that include a topic that the user 
> does not have DESCRIBE access to.
>       - A user should not receive a TopicAuthorizationException when they do 
> not have DESCRIBE access to a topic or the cluster.
>      - The client should not maintain and expose a list of unauthorized 
> topics in org.apache.kafka.common.Cluster. 
> Other changes may be required that are not listed here. Further analysis is 
> needed. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to