Consider using config for storing casbin configuration

[kibertoad]

As per https://github.com/fastify/fastify/blob/master/docs/Routes.md#options:

config: object used to store custom configuration.

[simoneb]

I didn't even know it existed!

[kibertoad]

@simoneb Also I didn't find a way to extend types for RouteOptions so that it would be aware of the new config values, maybe you know how?

[simoneb]

If you mean extending a type described elsewhere, I'm not a ts guru but this is how I've done it in the past https://github.com/simoneb/axios-hooks/blob/master/test/axios.d.ts

[kibertoad]

@simoneb Well, in fastify-casbin there is already this:

declare module 'fastify' {
  interface FastifyInstance {
    casbin: Enforcer & {
      casbinJsGetPermissionForUser(user: string): Promise<string>
    }
  }
}

But

declare module 'fastify' {
  interface RouteOptions{
    casbin: boolean
  }
}

did not work for me, so I wonder if I do it wrong or extend the wrong type.

[simoneb]

From what I can see RouteOptions is exported from https://github.com/fastify/fastify/blob/master/fastify.d.ts#L143 and defined https://github.com/fastify/fastify/blob/b8122e24c3329c1a84f4fe0625743fe55286c11f/types/route.d.ts#L102. So I would think what you posted works, but maybe the generics are playing a part into why it is not working

[kibertoad]

@simoneb Apparently needed to extend RouteShorthandOptions.