Skip to content

https-dns-proxy: update to 2026.03.18-2#29294

Merged
commodo merged 1 commit into
openwrt:masterfrom
stangri:master-https-dns-proxy
May 6, 2026
Merged

https-dns-proxy: update to 2026.03.18-2#29294
commodo merged 1 commit into
openwrt:masterfrom
stangri:master-https-dns-proxy

Conversation

@stangri
Copy link
Copy Markdown
Member

@stangri stangri commented May 4, 2026

Maintainer: me
Compile tested: x86_64, Dell EMC Edge620, OpenWrt 25.12.1
Run tested: x86_64, Dell EMC Edge620, OpenWrt 25.12.1

Description:
Fix nftables rule directory creation

Makefile:

  • Bump PKG_RELEASE to 2.

files/etc/init.d/https-dns-proxy:

  • Add 'mkdir -p' before writing nftables rules to ensure the parent
    directory exists. This fixes an issue where the directory might not
    exist on initial installation, causing errors.

tests/run_tests.sh:

  • Add comprehensive regression tests for notrack_nft.
  • Mock 'nft' to track invocations and control return codes for testing.
  • Patch 'NOTRACK_NFT_FILE' to a test-specific path for isolated testing.
  • Verify 'notrack_nft' correctly creates the parent directory if missing.
  • Test content of generated nftables snippet, idempotence, and removal

Maintainer: me
Compile tested: x86_64, Dell EMC Edge620, OpenWrt 25.12.1
Run tested: x86_64, Dell EMC Edge620, OpenWrt 25.12.1

Description:
Fix nftables rule directory creation

  - Bump PKG_RELEASE to 2.

files/etc/init.d/https-dns-proxy:
  - Add 'mkdir -p' before writing nftables rules to ensure the parent
    directory exists. This fixes an issue where the directory might not
    exist on initial installation, causing errors.

tests/run_tests.sh:
  - Add comprehensive regression tests for notrack_nft.
  - Mock 'nft' to track invocations and control return codes for testing.
  - Patch 'NOTRACK_NFT_FILE' to a test-specific path for isolated testing.
  - Verify 'notrack_nft' correctly creates the parent directory if missing.
  - Test content of generated nftables snippet, idempotence, and removal.

Signed-off-by: Stan Grishin <stangri@melmac.ca>
@commodo
Copy link
Copy Markdown
Contributor

commodo commented May 5, 2026

Does this fix #29282 ?

@commodo commodo mentioned this pull request May 6, 2026
3 tasks
@egc112
Copy link
Copy Markdown
Contributor

egc112 commented May 6, 2026

Does this fix #29282 ?

The main problem is already fixed in 81f0ef4
There still could be an error raised but that was not stopping the firewall from loading but anyway that should also be solved after this PR is committed

@commodo commodo merged commit 97b710a into openwrt:master May 6, 2026
7 of 12 checks passed
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.

3 participants