Simone Busoli
Simone Busoli
@dependabot rebase
ok so the PR was already up to date with master, and the problem is that the code needs to change to reflect that changes made in undici. @elmarcusec if...
Before we release what's in master, which has already bumped node-lru-cache to the latest release, we also need to drop compatibility with Node < 20, which I did in #310....
This would indeed be useful. Any news on whether this is going to be available in some future releases?
Thanks for reporting, what were you using in your previous flow to do the verification and which didn't exhibit any performance issues?
@imdavidmin it seems @marcin-iwanow is suggesting to switch to `jwks-rsa`. Shall we do a quick experimento to see if we notice any performance boost by doing the switch?
@marcin-iwanow can you provide a repro please?
Closing as we're missing a repro and we're unable to reproduce. Feel free to reopen in case you can reproduce the issue. Thanks @imdavidmin for looking into it.
I don't think we can safely merge this change, precisely because, as observed, this behavior is intentional. The change is straightforward, but before we even change anything, we need to...
Thanks for looking into this @AlessioScarfone . Based on your findings, I'm going to close this issue as the solution to this problem lies in user space.