search

GravlLift / halo-infinite-api

An NPM package for interacting with the Halo Infinite API.
5 stars 2 forks source link

Dependency "cockatiel" does not work with Cloudflare Workers #3

Open davidhouweling opened 3 weeks ago

davidhouweling commented 3 weeks ago

Context

I'm working on creating an app using Cloudflare Wworkers and there is a collision with AbortController within cocatiel.

I could try and raise it with the owners there but i thought it might be quicker to start here.

I understand that it is presently used as a retry mechanism, I wanted to understand what the need for it was and whether you would be okay with dropping it, or perhaps doing a rudimentary retry without the use of cocatiel?

I do have a working approach locally where I've removed cocatiel but it no longer has the retry mechanism.

Thoughts on having me raise a PR to drop it?

GravlLift commented 3 weeks ago

I don't think I want to remove retry functionality entirely, but I could probably be convinced to make it configurable so that the package can be used without it.

Maybe you can link me to a reproduction example of what you're facing? It doesn't have to be your full project, a minimal example would work too.