JonLuca De Caro comments

Results 80 comments of


                                            JonLuca De Caro

NextAuth v5 `auth(req, res)` fails in `pages/` API route

Hey @balazsorban44 it looks like the issue was first introduced here https://github.com/nextauthjs/next-auth/commit/65aa467c0e02c58fc37aa7559038102d18f7f3f0 at this file https://github.com/nextauthjs/next-auth/blame/main/packages/next-auth/src/lib/index.ts#L2 Is it intentional that next-auth v5 should no longer work with pages router?

[Bug]: ServiceWorker cannot cache custom scheme URLs

@MarshallOfSound I think that it's relatively safe to add cacheing of alternate handlers specific to electron. The protocol handler already registers schemas as privileged - it would be useful to...

Calling dereference mutates input schema, while example does not indicate this

Great idea, included this as a flag in the latest version to preserve the historical behavior. I also updated the readme to better reflect the mutation in place

Urlencoding of `$defs`

I dont believe this is happening anymore on latest - let me know with an example if it is

Convert relative refs when resolving an external schema

I'm open to a PR here but would need to update it to pass the tests and to use the new typescript repo. Closing this for now, feel free to...

Convert relative refs when resolving an external schema

It's this repo - all the code was migrated to TS ~8 months ago

#226: fix for external-from-external dependencies (exposed by addition of dereferencedCache)

I'm closing stale PRs - happy to merge in a new PR if it's rebased

References to references resolve relative to self, not baseUrl

https://github.com/APIDevTools/json-schema-ref-parser/pull/305/files

MissingPointerError: Token "ABC" does not exist.

Fixed in 11.3.1

Add the friendlyPath argument to $Refs::_resolve

AFAICT a refactor in the past changed this behavior. You'll get a `MissingPointerException` if the dereferencing hits a missing reference, and a manual call to `.get` on the reference will...