Dylan Vanmali
Dylan Vanmali
@cubic-dev-ai Please review
@Bekacru @himself65 PR is ready for review. I marked important concerns in the PR with notes to help you review at your own pace. Otherwise its feature complete :)
Moving back into draft to improve consent flow as mentioned by @Berndwl.
@smndtrl I patched a fix in #4237 but it hasn't been resolved. TLDR the array types are not stored on the database as arrays but instead as strings. I consider...
> A question that came up while evaluating this was: How can we get a auth.api.getSession for opaque or JWT based session tokens from this change? We're trying to have...
@smndtrl Happy to help :) For MCP clients, they **must** support the [resource parameter](https://modelcontextprotocol.io/specification/draft/basic/authorization#resource-parameter-implementation). For OAuth clients, they do not need to support the resource parameter, it is optional. In...
@smndtrl You should be able to use the `genericOauth` plugin with `tokenUrlParams: { resource: validAudience }`. Generic OAuth will create a user on your database and add a db session...
@Gittified can you add a test for this please
@Gittified sounds great! Looking forward to it! Zod was also upgraded recently on canary, if you rebase, you'll see the changes 👍