Method wildcard option does not work

[kpdecker]

var Hapi = require('hapi');

var server = Hapi.createServer('localhost', 8000);

server.route({
  method: '*',
  path: '/log',
  handler: function(request, reply) {
    reply({foo: 'bar'});
  }
});

server.start();

Returns a 404 error when curled.

[hueniverse]

Not seeing it.

[kpdecker]

This must be the missing piece.

server.route({
  method: 'GET',
  path: '/{path*}',
  handler: {
    directory: {
      path: 'static'
    }
  }
});

An explicit GET handler exists generically and is given precedence. Makes more sense now but seems less usable as the GET handler needs to go to * or need to expand out the log handler for each method that needs to be supported.

[hueniverse]

Literal verbs get precedence over wildcards:

https://github.com/spumko/hapi/blob/master/lib/router.js#L49-L56

I agree this is not ideal but not sure how to adjust it. Going back and forth between wildcard and verb-specific will degrade performance for the much more common use cases. It will require looking in both tables and if two matches found, comparing their paths for the more specific one.

I am inclined to simply document this and suggest you avoid using '*', but instead explicitly add the routes you need.

[kpdecker]

I think that is fair if documented.

Would you consider allowing an array of methods for the method parameter? Effectively would be syntax sugar for whatever construct that you would need to do to otherwise reduce code duplication if you need to support multiple methods for the same route and handler.

I.e.

server.route({
  method: ['GET', 'POST'],
  path: '/log',
  handler: function(request, reply) {
    reply({foo: 'bar'});
  }
});

[hueniverse]

Done.