HTTP/1.0 requests without a host header must be handled by the router

tsegismont commented 3 months ago

This is a regression introduced in https://github.com/vert-x3/vertx-web/pull/2612

Can be reproduced with:

package io.vertx.ext.web;

import io.vertx.core.AbstractVerticle;
import io.vertx.core.Promise;
import io.vertx.core.Vertx;

public class TestVerticle extends AbstractVerticle {

  public static void main(String[] args) {
    Vertx vertx = Vertx.vertx();
    vertx.deployVerticle(new TestVerticle())
      .onSuccess(id -> System.out.println("Deployed verticle " + id))
      .onFailure(Throwable::printStackTrace);
  }

  @Override
  public void start(Promise<Void> startPromise) throws Exception {
    Router router = Router.router(vertx);

    router.get().handler(rc -> {
      System.out.println("rc.request().authority() = " + rc.request().authority());
      rc.response().end("Hello World");
    });

    vertx.createHttpServer()
      .requestHandler(router)
      .listen(8080)
      .<Void>mapEmpty()
      .onComplete(startPromise);
  }
}

And then:

curl -v --http1.0  -H 'Host:'  localhost:8080

You will see something like this:

* Host localhost:8080 was resolved.
* IPv6: ::1
* IPv4: 127.0.0.1
*   Trying [::1]:8080...
* Connected to localhost (::1) port 8080
> GET / HTTP/1.0
> User-Agent: curl/8.6.0
> Accept: */*
> 
* HTTP 1.0, assume close after body
< HTTP/1.0 400 Bad Request
< content-length: 11
< 
* Closing connection
Bad Request

And the application logs to the console: