Skip to content

add doc for the new apache bouncer - #1202

Merged
buixor merged 10 commits into
mainfrom
mod_crowdsec-v1.0
Oct 26, 2026
Merged

buixor merged 10 commits into
mainfrom
mod_crowdsec-v1.0

Conversation

@buixor

@buixor buixor commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI balanced review requested due to automatic review settings October 22, 2026 13:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@aws-amplify-eu-west-1

Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-1202.d1to60jd2gb6y6.amplifyapp.com

@rr404 rr404 left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - just one P2 note:

in the "Size the cache for your decision count" We could mention that we recommand at least 100k IPs
And are we sure 100k = 6MB and 1M 128MB ?

@sabban sabban left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some comments

@buixor
buixor requested a review from sabban October 23, 2026 14:24

@sabban sabban left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@buixor
buixor merged commit 520a25e into main Oct 26, 2026
4 checks passed
@buixor
buixor deleted the mod_crowdsec-v1.0 branch October 26, 2026 12:08
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.

5 participants