Wildcard Method Requests

Description

The current method implementations don’t support catch-all endpoints.

Requested Syntax

const server = require('server');
const { status } = server.reply;
const { get } = server.router;

server(
  // This should get invoked for every request
  // An alternative syntax would be to support Regular Expressions.
  get('*', (ctx) => {
    return status(200);
  })
);

Current Workaround

const server = require('server');
const { status } = server.reply;

server(
  (ctx) => {
    if (ctx.method !== 'GET') return;

    // Do super cool api stuff
    return status(200);
  })
);

Relevant Code

The following code doesn’t support wildcards / catch-all endpoints.

https://github.com/franciscop/server/blob/dd1ad51bcd1cd80f64ff119e75edb51d3b6acbd9/router/generic.js#L21-L23

[teologov]

Hi guys. Yes, I should probably consider to remove the asterisk and migrate the module to version 2.x of path-to-regexp.

[franciscop]

I upgraded server to 1.0.7, please @nick-woodward let me know if this fixes the issue for you. I fixed the path-to-regexp version and hardcoded path-to-regexp-wrap. So it should install the right version and work as expected with both yarn and npm.

For version 2.0 of server I don’t know if we will depend on express or not; if we do, it’ll be very likely 5.0. I close this as I am really not sure what will happen by then and it’s still quite far away.