Matteo Collina comments

Results 3472 comments of


                                            Matteo Collina

Added Queue problem and empty setup/solution.

@juliangruber yeah, that's my point. But we want named queues/lists/whatever. So you think it's better just to prefix the keys, right?

Added Queue problem and empty setup/solution.

Here is a much better implementation of the queue, it solves some issues, such as concurrent push and shifts. https://github.com/mcollina/level-queue-type What I think it's missing is a "know more" text...

Added Queue problem and empty setup/solution.

@juliangruber thanks! done!

Added Queue problem and empty setup/solution.

This is a bit old, maybe we can add this in? :)

Typescript complication error when running tests using Vitest

I think that would be an amazing fix. However ts-node could not be there, so we must catch those error specifically and provide some instructions. Would you like to send...

Typescript complication error when running tests using Vitest

Unfortunately all those tools needs special handling in this library to work :(. I don't know if there is a way to handle it otherwise. PRs are welcomed!

Typescript complication error when running tests using Vitest

Unfortunately their ways of doing things is different, specifically when targeting ESM.

lib/util.js security feature issue

(my bad, I thought this was about fast-proxy)

lib/util.js security feature issue

As the error tells you, `rewritePrefix` __must__ be a path, not a full URL.

lib/util.js security feature issue

I would recommend to _not_ use fast-proxy with fastify. Use fastify-http-proxy or fastify-reply-from. fast-proxy is not supported with fastify.