Uploaded image for project: 'Kafka'
  1. Kafka
  2. KAFKA-10231

Broken Kafka Connect node to node communication if invalid hostname is in place

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Patch Available
    • Minor
    • Resolution: Unresolved
    • 2.4.0, 2.3.1, 2.5.0, 2.4.1
    • None
    • connect
    • None

    Description

      As a Kafka Connect operator I would expect a more definitive error when the internal node to node communication can't happen.

      If the hostname contains an invalid character according to the RFC1123 section 2.1, the error raised by the Kafka Connect worker node look like:

       

      [2020-06-30 10:38:49,990] ERROR Uncaught exception in REST call to /connectors (org.apache.kafka.connect.runtime.rest.errors.ConnectExceptionMapper)java.lang.IllegalArgumentException: Invalid URI host: null (authority: kafka_connect-0.dev-2:8083)        at org.eclipse.jetty.client.HttpClient.checkHost(HttpClient.java:506)        at org.eclipse.jetty.client.HttpClient.newHttpRequest(HttpClient.java:491)        at org.eclipse.jetty.client.HttpClient.newRequest(HttpClient.java:449)        at org.eclipse.jetty.client.HttpClient.newRequest(HttpClient.java:438)        at org.apache.kafka.connect.runtime.rest.RestClient.httpRequest(RestClient.java:83)        at org.apache.kafka.connect.runtime.rest.resources.ConnectorsResource.completeOrForwardRequest(ConnectorsResource.java:309)        at org.apache.kafka.connect.runtime.rest.resources.ConnectorsResource.createConnector(ConnectorsResource.java:138)

       

      it would be much nicer for operators that such situations are detected in similar, or improved, version as the JVM is doing in the IDN class].

       

      Attachments

        Issue Links

          Activity

            People

              kpatelatwork Kalpesh Patel
              purbon Pere Urbon-Bayes
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated: