Skip to content

Commit 5e4b311

Browse files
authored
Add NGINX App Protect DoS to the NAP uninstall tasks (#148)
1 parent 47eb502 commit 5e4b311

File tree

2 files changed

+14
-12
lines changed

2 files changed

+14
-12
lines changed

CHANGELOG.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,8 @@ Refactor how `nginx_app_protect_*_policy_file*` variables work. You can now spec
88

99
BUG FIXES:
1010

11-
Fix instances of `nginx_app_protect_license_status` being incorrectly set as `nginx_license_status` instead.
11+
* Fix instances of `nginx_app_protect_license_status` being incorrectly set as `nginx_license_status` instead.
12+
* Add NGINX App Protect DoS to the NAP uninstall tasks.
1213

1314
## 0.6.2 (October 25, 2021)
1415

tasks/main.yml

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,14 @@
3434

3535
- name: Set up signing keys
3636
include_tasks: "{{ role_path }}/tasks/common/keys/setup-keys.yml"
37-
when: nginx_app_protect_waf_state != "absent"
38-
or nginx_app_protect_install_signatures | bool
37+
when: nginx_app_protect_install_signatures | bool
3938
or nginx_app_protect_install_threat_campaigns | bool
4039
tags: nginx_app_protect_key
4140

4241
- name: Set up license
4342
include_tasks: "{{ role_path }}/tasks/common/install/setup-license.yml"
4443
when:
45-
- nginx_app_protect_waf_state != "absent"
46-
or nginx_app_protect_install_signatures | bool
44+
- nginx_app_protect_install_signatures | bool
4745
or nginx_app_protect_install_threat_campaigns | bool
4846
- nginx_app_protect_setup_license | bool
4947
tags: nginx_app_protect_setup_license
@@ -68,18 +66,21 @@
6866
when: nginx_app_protect_configure | bool
6967
tags: nginx_app_protect_configure
7068
when: nginx_app_protect_waf_state != "absent"
69+
or nginx_app_protect_dos_state != "absent"
7170

7271
- name: Ensure NGINX App Protect is running
7372
meta: flush_handlers
7473

7574
- name: Remove NGINX App Protect
7675
block:
77-
- name: Remove NGINX App Protect package
78-
include_tasks: "{{ role_path }}/tasks/waf/install/install-app-protect-waf.yml"
79-
tags: nginx_app_protect_remove
76+
- name: Remove NGINX App Protect WAF package
77+
include_tasks: "{{ role_path }}/tasks/waf/install-app-protect-waf.yml"
78+
when: nginx_app_protect_waf_state == "absent"
79+
tags: nginx_app_protect_waf_remove
8080

81-
- name: Disable NGINX App Protect config
82-
include_tasks: "{{ role_path }}/tasks/waf/config/configure-app-protect-waf.yml"
83-
when: nginx_app_protect_configure | bool
84-
tags: nginx_app_protect_remove_config
81+
- name: Remove NGINX App Protect DoS package
82+
include_tasks: "{{ role_path }}/tasks/dos/install-app-protect-waf.yml"
83+
when: nginx_app_protect_dos_state == "absent"
84+
tags: nginx_app_protect_dos_remove
8585
when: nginx_app_protect_waf_state == "absent"
86+
or nginx_app_protect_dos_state == "absent"

0 commit comments

Comments
 (0)