[ https://issues.apache.org/jira/browse/KAFKA-4930?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15943621#comment-15943621 ]
Gwen Shapira commented on KAFKA-4930: ------------------------------------- [~sliebau] I like your validName approach. Can you create a PR? It is much easier for me to review that way :) > Connect Rest API allows creating connectors with an empty name > -------------------------------------------------------------- > > Key: KAFKA-4930 > URL: https://issues.apache.org/jira/browse/KAFKA-4930 > Project: Kafka > Issue Type: Bug > Components: KafkaConnect > Affects Versions: 0.10.2.0 > Reporter: Sönke Liebau > Priority: Minor > > The Connect Rest API allows to deploy connectors with an empty name field, > which then cannot be removed through the api. > Sending the following request: > {code} > { > "name": "", > "config": { > "connector.class": > "org.apache.kafka.connect.tools.MockSourceConnector", > "tasks.max": "1", > "topics": "test-topic" > > } > } > {code} > Results in a connector being deployed which can be seen in the list of > connectors: > {code} > [ > "", > "testconnector" > ]{code} > But cannot be removed via a DELETE call, as the api thinks we are trying to > delete the /connectors endpoint and declines the request. > I don't think there is a valid case for the connector name to be empty so > perhaps we should add a check for this. I am happy to work on this. -- This message was sent by Atlassian JIRA (v6.3.15#6346)