support express v5

[JYbill]

Goals

none

Non-Goals

none

Background

none

Proposal

• in @auth/express, source code

function getBasePath(req: e.Request) {
  // ⚠️ req.params[0]
 // in v5-, "/auth/*", /auth/signin will get "signin"，
 // but in v5, must be defined "/auth/*path", req.params:  { path: [ 'signin' ] }, the `req.params.path[0]` is right
 // so, default in (express/nestjs)v5 will meet [auth][error] UnknownAction: Cannot parse action at /auth/signin
  return req.baseUrl.split(req.params[0])[0].replace(/\/$/, "")
}

we should to agree on a wildcard variable, maybe "/auth/*params" or others.

[JYbill]

in fact, I've modified the source code to make it work by default

export function ExpressAuth(config: ExpressAuthConfig) {
  return async (req: e.Request, res: e.Response, next: e.NextFunction) => {
    e.json()(req, res, async (err) => {
      if (err) return next(err)
      e.urlencoded({ extended: true })(req, res, async (err) => {
        if (err) return next(err)
        try {
          // config.basePath = getBasePath(req) // 💡 setEnvDefaults will set basePath = "/auth"
          setEnvDefaults(process.env, config)
          await toExpressResponse(await Auth(toWebRequest(req), config), res)
          if (!res.headersSent) next()
        } catch (error) {
          next(error)
        }
      })
    })
  }
}