Skip to content

Conversation

bdc34
Copy link

@bdc34 bdc34 commented May 8, 2024

The bouncer ACLCollection state would get out of sync with what was in fastly.

If a HTTP request to fastly failed due to a 429, the process would die and the state cache file could get out of sync.

This PR fixes 4 things:

  1. it adds a refresh all ACLs mode that is run at process start and on SIGHUP
  2. It alters order of operations when ACLs are updated at fastly so the local state is not updated until faslty successfully updates
  3. It adds an auto-repair mode. If fastly returns a 400 response to an ACL change, the ACL will be updated and the change will be run again.
  4. Bumps the batch size and ACL size from 100 to 1000.

bdc34 added 2 commits May 8, 2024 14:49
This facilitates disabling 'captcha' VCL and ACL setup.

The recaptcha config does not work with my config at fastly. In particular
I do not have a backend called F_Host_1.
Adds ability to use `supported_actions` in initial config
@LaurenceJJones LaurenceJJones self-requested a review September 3, 2024 13:43
@LaurenceJJones
Copy link

Hey 👋🏻

Thank you for opening a pull request!

Apologizes for the delay in responding. I will check out the changes and keep you posted.

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

Successfully merging this pull request may close these issues.

2 participants