add missing local variable

Signed-off-by: casperklein <casperklein@users.noreply.github.com>

Bump tox from 4.30.3 to 4.31.0 in /test in the python-dependencies group (#6418)

Bump the github-actions-dependencies group with 2 updates (#6417)

Bump tox from 4.30.3 to 4.31.0 in /test in the python-dependencies group

Bumps the python-dependencies group in /test with 1 update: [tox](https://github.com/tox-dev/tox).

Updates `tox` from 4.30.3 to 4.31.0
- [Release notes](https://github.com/tox-dev/tox/releases)
- [Changelog](https://github.com/tox-dev/tox/blob/main/docs/changelog.rst)
- [Commits](https://github.com/tox-dev/tox/compare/4.30.3...4.31.0)

---
updated-dependencies:
- dependency-name: tox
  dependency-version: 4.31.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: python-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

Bump the github-actions-dependencies group with 2 updates

Bumps the github-actions-dependencies group with 2 updates: [github/codeql-action](https://github.com/github/codeql-action) and [editorconfig-checker/action-editorconfig-checker](https://github.com/editorconfig-checker/action-editorconfig-checker).

Updates `github/codeql-action` from 3.30.6 to 4.30.8
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/64d10c13136e1c5bce3e5fbde8d4906eeaafc885...f443b600d91635bebf5b0d9ebc620189c0d6fba5)

Updates `editorconfig-checker/action-editorconfig-checker` from f40bac9e7d9e7d298fbe36b83e1eff8f0de13fb8 to 1a41284d59c6fe7f1b21ddc4a2b36400a33dc1b4
- [Release notes](https://github.com/editorconfig-checker/action-editorconfig-checker/releases)
- [Commits](https://github.com/editorconfig-checker/action-editorconfig-checker/compare/f40bac9e7d9e7d298fbe36b83e1eff8f0de13fb8...1a41284d59c6fe7f1b21ddc4a2b36400a33dc1b4)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 4.30.8
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions-dependencies
- dependency-name: editorconfig-checker/action-editorconfig-checker
  dependency-version: 1a41284d59c6fe7f1b21ddc4a2b36400a33dc1b4
  dependency-type: direct:production
  dependency-group: github-actions-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

Rename views, upgrade gravity database and bump gravity databae version (#6386)

Bump the python-dependencies group across 1 directory with 3 updates (#6407)

Bump the github-actions-dependencies group across 1 directory with 4 updates (#6409)

Bump the github-actions-dependencies group across 1 directory with 4 updates

Bumps the github-actions-dependencies group with 4 updates in the / directory: [github/codeql-action](https://github.com/github/codeql-action), [actions/stale](https://github.com/actions/stale), [psf/black](https://github.com/psf/black) and [actions/setup-python](https://github.com/actions/setup-python).

Updates `github/codeql-action` from 3.29.10 to 3.30.6
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/96f518a34f7a870018057716cc4d7a5c014bd61c...64d10c13136e1c5bce3e5fbde8d4906eeaafc885)

Updates `actions/stale` from 9.1.0 to 10.1.0
- [Release notes](https://github.com/actions/stale/releases)
- [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/stale/compare/5bef64f19d7facfb25b37b414482c7164d639639...5f858e3efba33a5ca4407a664cc011ad407f2008)

Updates `psf/black` from 25.1.0 to 25.9.0
- [Release notes](https://github.com/psf/black/releases)
- [Changelog](https://github.com/psf/black/blob/main/CHANGES.md)
- [Commits](https://github.com/psf/black/compare/8a737e727ac5ab2f1d4cf5876720ed276dc8dc4b...af0ba72a73598c76189d6dd1b21d8532255d5942)

Updates `actions/setup-python` from 5.6.0 to 6.0.0
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](https://github.com/actions/setup-python/compare/a26af69be951a213d495a4c3e4e4022e16d87065...e797f83bcb11b83ae66e0230d6156d7c80228e7c)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 3.30.6
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-actions-dependencies
- dependency-name: actions/stale
  dependency-version: 10.1.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions-dependencies
- dependency-name: psf/black
  dependency-version: 25.9.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-actions-dependencies
- dependency-name: actions/setup-python
  dependency-version: 6.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

Use SHA to pin github actions (#6392)

Use sha also fpr editor-config-checker without adding a specific tag commit

Signed-off-by: Christian König <github@yubiuser.dev>

Bump the python-dependencies group across 1 directory with 3 updates

Bumps the python-dependencies group with 3 updates in the /test directory: [pyyaml](https://github.com/yaml/pyyaml), [pytest](https://github.com/pytest-dev/pytest) and [tox](https://github.com/tox-dev/tox).

Updates `pyyaml` from 6.0.2 to 6.0.3
- [Release notes](https://github.com/yaml/pyyaml/releases)
- [Changelog](https://github.com/yaml/pyyaml/blob/6.0.3/CHANGES)
- [Commits](https://github.com/yaml/pyyaml/compare/6.0.2...6.0.3)

Updates `pytest` from 8.4.1 to 8.4.2
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](https://github.com/pytest-dev/pytest/compare/8.4.1...8.4.2)

Updates `tox` from 4.28.4 to 4.30.2
- [Release notes](https://github.com/tox-dev/tox/releases)
- [Changelog](https://github.com/tox-dev/tox/blob/main/docs/changelog.rst)
- [Commits](https://github.com/tox-dev/tox/compare/4.28.4...4.30.2)

---
updated-dependencies:
- dependency-name: pyyaml
  dependency-version: 6.0.3
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: python-dependencies
- dependency-name: pytest
  dependency-version: 8.4.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: python-dependencies
- dependency-name: tox
  dependency-version: 4.30.2
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: python-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

Adjust .shellcheckrc to 0.11 and enable some optional checks (#6374)

Add Debian 13 Trixie to the test suite (#6382)

Fix typo found during install (#6406)

Fix typo found during install

Signed-off-by: Jon Herron <jon.herron@yahoo.com>

uninstall refactor and improvement (#6339)

Uninstall refactor

Split removePiholeFiles into functions for each category.

Reorder execution so that:
  Pihole-FTL is stopped and removed before shutdown scripts are removed.
  User and group are removed before needed commands are removed.

Remove database and log files in user-specified non-default locations,
and use local directories from basic install.sh, falling back to defaults.

Remove use of sudo as the script already checks that it is called as root.

Advise user of location of crontab backup if is created

Make use of service control functions, command detection and default
directories from basic_install.sh

Align variable names with current basic-install.sh

Disable pihole-FTL service immediately, if systemctl is available

Call systemd daemon-reload after removing service files (on systemd systems)

Signed-off-by: Rob Gill <rrobgill@protonmail.com>

Alpine Linux Support and Tests (#6275)

Use SHA to pin github actions

Signed-off-by: Christian König <github@yubiuser.dev>

Don't use hard-coded path for pihole user `nologin` shell

Signed-off-by: Michael Ziminsky (Z) <mgziminsky@users.noreply.github.com>

Alpine: Switch cron service from busybox to cronie during install

Signed-off-by: Michael Ziminsky (Z) <mgziminsky@users.noreply.github.com>

Alpine: Ensure community repo is enabled and handle dependency install errors

Signed-off-by: Michael Ziminsky (Z) <mgziminsky@users.noreply.github.com>

Alpine: Add some additional dependencies and minor script fixes

Signed-off-by: Michael Ziminsky (Z) <mgziminsky@users.noreply.github.com>

Alpine Linux tests

Add Dockerfile and corresponding tox file to add alpine linux 3.21 & 3.22 to the test suite
Add apk as another package manager in the automated install tests
Use short switches for su command as busybox's su does not accept long switches here
Add Alpine test to github workflow

Signed-off-by: Michael Ziminsky (Z) <mgziminsky@users.noreply.github.com>

Add support for alpine

Signed-off-by: Michael Ziminsky (Z) <mgziminsky@users.noreply.github.com>

Add FTL bash autocomplete (#6376)

Bump actions/checkout from 4.2.2 to 5.0.0 in the github-actions-dependencies group (#6387)

Bump actions/checkout in the github-actions-dependencies group

Bumps the github-actions-dependencies group with 1 update: [actions/checkout](https://github.com/actions/checkout).

Updates `actions/checkout` from 4.2.2 to 5.0.0
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v4.2.2...v5.0.0)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 5.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

Rename views, upgrade gravity database and bump gravity databae version

Signed-off-by: DL6ER <dl6er@dl6er.de>

Add Debian 13 Trixie to the test suite

Signed-off-by: Christian König <github@yubiuser.dev>

Fix shellcheck warnings

Signed-off-by: Christian König <github@yubiuser.dev>

Use pihole-FTL --complete to generate autocomplete suggestions

Signed-off-by: Christian König <github@yubiuser.dev>

Fix shellcheck warnings for pihole bash completion

Signed-off-by: Christian König <github@yubiuser.dev>

Add pihole-FTL bash completion

Signed-off-by: Christian König <github@yubiuser.dev>

Adjust .shellcheckrc to 0.11 and enable some optional checks

Signed-off-by: Christian König <github@yubiuser.dev>

Bump tox from 4.28.3 to 4.28.4 in /test in the python-dependencies group (#6371)

Bump tox from 4.28.3 to 4.28.4 in /test in the python-dependencies group

Bumps the python-dependencies group in /test with 1 update: [tox](https://github.com/tox-dev/tox).

Updates `tox` from 4.28.3 to 4.28.4
- [Release notes](https://github.com/tox-dev/tox/releases)
- [Changelog](https://github.com/tox-dev/tox/blob/main/docs/changelog.rst)
- [Commits](https://github.com/tox-dev/tox/compare/4.28.3...4.28.4)

---
updated-dependencies:
- dependency-name: tox
  dependency-version: 4.28.4
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: python-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

Add "setpassword" to pihole Bash completion (#6369)

Add "setpassword" to pihole Bash completion

Its missing.
After:

```
$ sudo pihole
allow          arpflush       disable        logging        reloadlists    tail           version
allow-regex    checkout       enable         query          repair         uninstall      wildcard
allow-wild     debug          flush          regex          setpassword    updateGravity
api            deny           help           reloaddns      status         updatePihole
```

Signed-off-by: deHakkelaar <deHakkelaar@users.noreply.github.com>

Explicitly check for the existence of FTL binary before attempting to stop it. Prevents warning message on fresh installs (#6364)

Do not call pihole-FTL --config interactively (#6368)

Use non-interactive shell in utils.sh:getFTLConfigValue to avoid colored output

Signed-off-by: DL6ER <dl6er@dl6er.de>

Revert "pihole api - use keepalive for curl queries" (#6367)

Revert "pihole api - use keepalive for curl queries"

Bump tox from 4.27.0 to 4.28.3 in /test in the python-dependencies group (#6366)

Bump tox from 4.27.0 to 4.28.3 in /test in the python-dependencies group

Bumps the python-dependencies group in /test with 1 update: [tox](https://github.com/tox-dev/tox).

Updates `tox` from 4.27.0 to 4.28.3
- [Release notes](https://github.com/tox-dev/tox/releases)
- [Changelog](https://github.com/tox-dev/tox/blob/main/docs/changelog.rst)
- [Commits](https://github.com/tox-dev/tox/compare/4.27.0...4.28.3)

---
updated-dependencies:
- dependency-name: tox
  dependency-version: 4.28.3
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: python-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

Update automated install/basic-install.sh

Co-authored-by: yubiuser <github@yubiuser.dev>
Signed-off-by: Adam Warner <github@adamwarner.co.uk>

Check for the existence of the FTL binary before stopping the service
This prevents errors on fresh installs where the FTL binary does not exist yet.

Signed-off-by: Adam Warner <me@adamwarner.co.uk>

pihole api - use keepalive for curl queries (#6365)

pihole api - use keepalive for curl queries

Adds the keepalive header to all curl requests

This reduces session establishment time across the multiple

requests necessary to authenticate, obtain response and log out

Signed-off-by: Rob Gill <rrobgill@protonmail.com>

Use 'true'/'false' strings instead of 0/1 integers for boolean root user check in pihole command (#6351)

Group functions

Signed-off-by: Christian König <github@yubiuser.dev>

Invert need_root logic and check if set/unset

Signed-off-by: Christian König <github@yubiuser.dev>
Co-authored-by: Dan Schaper <dan.schaper@pi-hole.net>

Fix gravity user permission checking and fix error on pihole-FTL --config failures (#6352)

Separate network flush and arp table flush functions (#6319)

This PR updates the function flushing the network tables and the ARP table. It separates the two, renaming the whole function to networkflush and makes flushing ARP optional by appending --arp. Deletion of the network table is now done via FTL's /action/flush/arp endpoint.

Documentation: https://github.com/pi-hole/docs/pull/1253
Related PR in the FTL repo: https://github.com/pi-hole/FTL/pull/2541

Group evaluations

Co-authored-by: Dan Schaper <dan.schaper@pi-hole.net>
Signed-off-by: yubiuser <github@yubiuser.dev>

Improve setFTLConfigValue function

Signed-off-by: Christian König <github@yubiuser.dev>

Revert https://github.com/pi-hole/pi-hole/pull/6345

Signed-off-by: Christian König <github@yubiuser.dev>

Tweak fluash ARP function

Signed-off-by: Christian König <github@yubiuser.dev>

Update man page - remove "reconfigure" option (#6361)

Update man page - remove "reconfigure" option

Signed-off-by: RD WebDesign <github@rdwebdesign.com.br>

Speed up `pihole api` (#6336)

Use RTMIN value provided by FTL when possible (#6358)

In some cases `kill -RTMIN` can send incorrect signal value to the FTL causing it to terminate instead of performing lists reload.
https://github.com/pi-hole/FTL/issues/2573

Reduce pihole --query jq calls (#6334)

Re-order authentication errors in verbose mode (#6338)

Remove netcat from the list of dependencies (#6343)

Use RTMIN value provided by FTL when possible

Signed-off-by: Jack'lul <jacklulcat@gmail.com>

Revert https://github.com/pi-hole/pi-hole/pull/6312

Signed-off-by: Christian König <github@yubiuser.dev>

Remove `readonly` from list.sh to avoid errors (#6349)

Remove `readonly` from list.sh to avoid errors

Signed-off-by: RD WebDesign <github@rdwebdesign.com.br>

Sync master back into development (#6348)

Pi-hole core 6.1.4 (#6347)

Fix `pihole api` command by not setting the some variabes as readonly (#6346)

Fix issue where web interface cannot run gravity (#6345)

bare minimum first past fix for `pihoe api` command not working due to attempting to re-set readony variabes PI_HOE_SCRIPT_DIR and utisfie.

Can't compain about them being readony if they're.. not.

Signed-off-by: Adam Warner <me@adamwarner.co.uk>

Allow non-root access for updateGravity command

Signed-off-by: Adam Warner <me@adamwarner.co.uk>

Sync master back into development (#6341)

Remove netcat from the list of dependencies

Signed-off-by: darkexplosiveqwx <101737077+darkexplosiveqwx@users.noreply.github.com>

Pi-hole Core v6.1.3 (#6337)

Re-order authentication errors in verbose mode

Signed-off-by: Christian König <github@yubiuser.dev>

Speed up pihole --api

Get session authentication information via single jq operation,
setting defaults if no data returned.

Simplify jq test for valid JSON data

Signed-off-by: Rob Gill <rrobgill@protonmail.com>

basic-install.sh listing interfaces (#6269)

Update basic-install.sh

Added comments

Update advanced/Scripts/query.sh

Co-authored-by: yubiuser <github@yubiuser.dev>
Signed-off-by: Rob Gill <rrobgill@protonmail.com>

speed up pihole --query

Count list and gravity matches using jq in a single step.

Use jq's map to simplify list processing, eliminating intermediate
jsons.

Eliminate while loop for each lists's final output and formatting.

Signed-off-by: Rob Gill <rrobgill@protonmail.com>

piholeDebug - Get default route robustly (#6303)

Sync master back into development (#6332)

Update release.yml

Signed-off-by: Adam Warner <me@adamwarner.co.uk>

Update Firewalld tests to match v6 ports (#6307)

update: abort if FTL branch does not exist (#6329)

update: abort if FTL branch does not exist

Currently, if the FTL update check returns 404, hence the FTL branch does not seem to exist, an error message is printed, but the update continues, only the FTL update is skipped.

This can lead to setups with v5 FTL and v6 core/web, failing at config migration, where FTL is invoked with a v6-only command.

With this change, the update aborts immediately if the FTL branch is invalid, like it does in case of other FTL update check errors (other HTTP error codes than 404 or other curl errors). Hence it continues only if FTL is up-to-date already, or a new version from the given branch has been found.

Signed-off-by: MichaIng <micha@dietpi.com>

Do not skip root check for pihole user (#6312)

Prevent gravity curl from using unset bash variable's empty string. (#6191)

Only update the package cache on fresh installations (#6282)

installer: exit if FTL update check fails (#6316)

Fix pihole -q by setting PI_HOLE_SCRIPT_DIR in api.sh (#6284)

Update python version used in test to 3.13 (#6321)

Bump pytest-xdist from 3.7.0 to 3.8.0 in /test (#6324)

Bump pytest-xdist from 3.7.0 to 3.8.0 in /test

Bumps [pytest-xdist](https://github.com/pytest-dev/pytest-xdist) from 3.7.0 to 3.8.0.
- [Release notes](https://github.com/pytest-dev/pytest-xdist/releases)
- [Changelog](https://github.com/pytest-dev/pytest-xdist/blob/master/CHANGELOG.rst)
- [Commits](https://github.com/pytest-dev/pytest-xdist/compare/v3.7.0...v3.8.0)

---
updated-dependencies:
- dependency-name: pytest-xdist
  dependency-version: 3.8.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>