kafka-jira mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sönke Liebau (JIRA) <j...@apache.org>
Subject [jira] [Commented] (KAFKA-4930) Connect Rest API allows creating connectors with an empty name
Date Fri, 30 Jun 2017 07:00:02 GMT

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

Sönke Liebau commented on KAFKA-4930:
-------------------------------------

I've updated the PR to resolve conflict that were caused by introducing the nonEmptyString
validator. As far as I can tell that validator is only being used in the transactional producer
config, so this PR should still apply as discussed previously.

> 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.4.14#64029)

Mime
View raw message