Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[cs-cloudflare-worker] Document an issue if zone has no A or AAAA #569

Open
LaurenceJJones opened this issue May 7, 2024 · 3 comments
Open

Comments

@LaurenceJJones
Copy link
Contributor

User was experiencing an issue when enabling the worker on all zones, and then the service was constantly restarting the issue was that a zone had no (A or AAAA to be confirmed) and was causing restart so most likely there an issue in the code, this should be handled.

For now we will update the documentation to showcase this.

@thomasgauvin
Copy link

Hi @LaurenceJJones, Thomas here! (PM for Cloudflare Workers KV)

We're noticing that there's a lot of unnecessary usage on Workers KV by various customers of CrowdSec and package crowdsec-cloudflare-worker-bouncer because of this constant restart.

This usage was previously underreported. We're going to be releasing improvements to our billing system which will accurately count this usage (previously this was undercounted by 10x).

Any chance you are planning on getting this fix in the docs + in the code (such as erroring instead of restarting) some time soon?

I also sent an email out earlier today to see how we can work together to issue a fix

@blotus
Copy link
Member

blotus commented Jul 9, 2024

Hello @thomasgauvin,

Thanks for the heads-up, and sorry for the inconvenience.

I've created another issue to track this: crowdsecurity/cs-cloudflare-worker-bouncer#41
I expect to have a fix ready fairly soon.

@thomasgauvin
Copy link

thomasgauvin commented Jul 9, 2024

Thanks @blotus, and great work on CrowdSec & the cd-cloudflare-worker-bouncer! Some customers also mentioned that their restarts may be related to another issue, such as crowdsecurity/cs-cloudflare-worker-bouncer#40 (comment) (which you seem to be aware of). Can you keep us updated on these fixes when they go out?

(if you need to get in touch directly, firstinitiallastname at cloudflare dot com, or via Twitter/LinkedIn)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants