Merge pull request 'refactor ♻️: Check for firewall file existence before debugging' (#51 ) from dev into main

Reviewed-on: #51
refactor ♻️: Check for firewall file existence before debugging
2026-03-01 20:05:15 +01:00 · 2026-03-01 20:03:52 +01:00
1 changed files with 1 additions and 1 deletions
--- a/tasks/fail2ban.yml
+++ b/tasks/fail2ban.yml
@@ -182,7 +182,7 @@
  ansible.builtin.debug:
    msg: >
      {{ fw_content }}
-  when: not pve_firewall_enabled
+  when: fw_stat.stat.exists | default(false)

 - name: fail2ban | Determine if firewall enabled
  ansible.builtin.set_fact:
Author	SHA1	Message	Date
Jose	0bfe448b34	Merge pull request 'refactor ♻️: Check for firewall file existence before debugging' (#51 ) from dev into main All checks were successful ansible-lint / Ansible Lint (push) Successful in 12s Details Gitleaks Scan / gitleaks (push) Successful in 4s Details Markdown Lint / markdown-lint (push) Successful in 6s Details Mark stale issues and pull requests / stale (push) Successful in 2s Details Reviewed-on: #51	2026-03-01 20:05:15 +01:00
Jose	38831f981a	refactor ♻️: Check for firewall file existence before debugging All checks were successful ansible-lint / Ansible Lint (push) Successful in 12s Details Gitleaks Scan / gitleaks (push) Successful in 4s Details Markdown Lint / markdown-lint (push) Successful in 5s Details ai-reviews / Review PR (pull_request) Successful in 11s Details PR check / Gitleaks (pull_request) Successful in 4s Details PR check / lint tests (pull_request) Successful in 15s Details PR check / labeler (pull_request) Successful in 2s Details PR check / handle_failures (pull_request) Has been skipped Details PR check / handle_success (pull_request) Successful in 2s Details Refactored the code to include a conditional check for the existence of the firewall file before proceeding with debugging. This ensures that the debugging process is only initiated when necessary, preventing unnecessary operations and potential errors.	2026-03-01 20:03:52 +01:00