[PR #558] [MERGED] Added dependency of crowdsec to traefik and relaxed its health check #825

New Issue

GiteaMirror · 2025-11-13T12:11:45-06:00

GiteaMirror commented

2025-11-13 12:11:45 -06:00

📋 Pull Request Information

Original PR: https://github.com/fosrl/pangolin/pull/558
Author: @TuncTaylan
Created: 4/21/2025
Status: ✅ Merged
Merged: 4/29/2025
Merged by: @oschwartz10612

Base: dev ← Head: main

📝 Commits (6)

c837899 Relaxed health check for crowdsec and dependance to traefik
181071e refactoring multiple used code parts, cleared some warnings and added more error checking
87915f2 correct formated the file
6fd1dbc some more warning about indirect requirements
419bacf check and add the service dependency of crowdsec to traefik
5d6e15b indentation fix

📊 Changes

5 files changed (+124 additions, -133 deletions)

View changed files

📝 install/config/crowdsec/docker-compose.yml (+3 -0)
📝 install/crowdsec.go (+64 -0)
📝 install/go.mod (+4 -3)
📝 install/go.sum (+1 -0)
📝 install/main.go (+52 -130)

📄 Description

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

Add dependancy on crowdsec for traefik and more relaxed health check for crowdsec, to prevent zombie traefik bouncers when starting the stack up.

Otherwise there are always "loose" or duplicate traefik bouncer plugins (check the last api pull dates):

docker exec -it crowdsec cscli bouncers list
──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
 Name                        IP Address  Valid  Last API pull         Type                             Version                                                                  Auth Type
──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
 traefik-bouncer             172.18.0.4  ✔️     2025-04-21T09:24:34Z  Crowdsec-Bouncer-Traefik-Plugin  1.X.X                                                                    api-key
 vps-firewall                172.18.0.1  ✔️     2025-04-21T09:32:16Z  crowdsec-firewall-bouncer        v0.0.31-debian-pragmatic-amd64-4b99c161b2c1837d76c5fa89e1df83803dfbcc87  api-key
 traefik-bouncer@172.18.0.3  172.18.0.3  ✔️     2025-04-18T20:43:20Z  Crowdsec-Bouncer-Traefik-Plugin  1.X.X                                                                    api-key
 traefik-bouncer@172.18.0.2  172.18.0.2  ✔️     2025-04-15T06:55:59Z  Crowdsec-Bouncer-Traefik-Plugin  1.X.X                                                                    api-key
──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

How to test?

Just run the installer and do a docker-compose.yaml comparison.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/fosrl/pangolin/pull/558 **Author:** [@TuncTaylan](https://github.com/TuncTaylan) **Created:** 4/21/2025 **Status:** ✅ Merged **Merged:** 4/29/2025 **Merged by:** [@oschwartz10612](https://github.com/oschwartz10612) **Base:** `dev` ← **Head:** `main` --- ### 📝 Commits (6) - [`c837899`](https://github.com/fosrl/pangolin/commit/c837899d82895bbd3007ad4d55c0ca54c54df2a5) Relaxed health check for crowdsec and dependance to traefik - [`181071e`](https://github.com/fosrl/pangolin/commit/181071e4f63b28d8e1ac3ecd198f87af4387a53f) refactoring multiple used code parts, cleared some warnings and added more error checking - [`87915f2`](https://github.com/fosrl/pangolin/commit/87915f29f6e7095baa3e1ceb0857b2cc17d17c8f) correct formated the file - [`6fd1dbc`](https://github.com/fosrl/pangolin/commit/6fd1dbc638a83e4a0a20459217f908c9f21e1bf3) some more warning about indirect requirements - [`419bacf`](https://github.com/fosrl/pangolin/commit/419bacf55ff6cd61ff29c4f8e61ac4c5549f8386) check and add the service dependency of crowdsec to traefik - [`5d6e15b`](https://github.com/fosrl/pangolin/commit/5d6e15b0d66b2fe70e82be1e1034591e5e516407) indentation fix ### 📊 Changes **5 files changed** (+124 additions, -133 deletions) <details> <summary>View changed files</summary> 📝 `install/config/crowdsec/docker-compose.yml` (+3 -0) 📝 `install/crowdsec.go` (+64 -0) 📝 `install/go.mod` (+4 -3) 📝 `install/go.sum` (+1 -0) 📝 `install/main.go` (+52 -130) </details> ### 📄 Description ## Community Contribution License Agreement By creating this pull request, I grant the project maintainers an unlimited, perpetual license to use, modify, and redistribute these contributions under any terms they choose, including both the AGPLv3 and the Fossorial Commercial license terms. I represent that I have the right to grant this license for all contributed content. ## Description Add dependancy on crowdsec for traefik and more relaxed health check for crowdsec, to prevent zombie traefik bouncers when starting the stack up. Otherwise there are always "loose" or duplicate traefik bouncer plugins (check the last api pull dates): ```bash docker exec -it crowdsec cscli bouncers list ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── Name IP Address Valid Last API pull Type Version Auth Type ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── traefik-bouncer 172.18.0.4 ✔️ 2025-04-21T09:24:34Z Crowdsec-Bouncer-Traefik-Plugin 1.X.X api-key vps-firewall 172.18.0.1 ✔️ 2025-04-21T09:32:16Z crowdsec-firewall-bouncer v0.0.31-debian-pragmatic-amd64-4b99c161b2c1837d76c5fa89e1df83803dfbcc87 api-key traefik-bouncer@172.18.0.3 172.18.0.3 ✔️ 2025-04-18T20:43:20Z Crowdsec-Bouncer-Traefik-Plugin 1.X.X api-key traefik-bouncer@172.18.0.2 172.18.0.2 ✔️ 2025-04-15T06:55:59Z Crowdsec-Bouncer-Traefik-Plugin 1.X.X api-key ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── ``` ## How to test? Just run the installer and do a docker-compose.yaml comparison. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2025-11-13 12:11:45 -06:00

GiteaMirror closed this issue