search

crowdsecurity / cs-cloudflare-bouncer

A CrowdSec Bouncer that syncs the decisions made by CrowdSec with CloudFlare's firewall. Manages multi user, multi account, multi zone setup. Supports IP, Country and AS scoped decisions.
MIT License
50 stars 7 forks source link

Broken cloudflare API #96

Closed sbs2001 closed 2 years ago

sbs2001 commented 2 years ago

CF seems to have changed the format to delete multiple filters at https://developers.cloudflare.com/firewall/api/cf-filters/delete/#delete-multiple-filters . The go lib which we use seems to call outdated endpoint. The error we get when deleting multiple filters

time="19-05-2022 22:46:10" level=fatal msg="HTTP status 400: invalid field value: 73d4d593bca54aa6a21f2dbfb261e81a,7631acc1987640f49ef3926440f07e43"

Update:

We also have another issue https://github.com/cloudflare/cloudflare-go/issues/909

sbs2001 commented 2 years ago

Tracked upstream at https://github.com/cloudflare/cloudflare-go/issues/903

sbs2001 commented 2 years ago

Fixed in upstream at https://github.com/cloudflare/cloudflare-go/pull/904

sbs2001 commented 2 years ago

Not closing since we have https://github.com/cloudflare/cloudflare-go/issues/909

sbs2001 commented 2 years ago

Should be fixed by https://github.com/crowdsecurity/cs-cloudflare-bouncer/pull/94/commits/5a6ee0a3a592fe377c89315b81056fde12b4538a