Lookup route during OPTIONS by using request.info.hostname. Closes #2930

[mtharrison]

During a normal request the route is looked up using the value of request.info.hostname:

const match = this.connection._router.route(this.method, this.path, this.info.hostname);

However during an OPTIONS request, it's currently being looked up via the value of request.headers.host:

const route = request.connection.match(method, request.path, request.headers.host);

These values are different when not using port 80, therfore this can cause the route not to be found correctly when there's a vhost setting on a route as per hapijs/glue#38 #2930.

[devinivy]

Thanks for tracking this down– looks good to me.

[lock]

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.