Resolve GitHub issue #45 in H.Vpn

[HavenDV]

Summary by CodeRabbit

• Bug Fixes

  • Fixed IPv4 subnet traffic blocking to correctly block network traffic instead of permitting it.

• Style

  • Applied whitespace and formatting adjustments across build configuration and project files.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request updates firewall session blocking behavior and applies formatting standardization across project files. The primary functional change modifies BlockSubNetworkV4 in SessionExtensions.cs to block traffic instead of permitting it, with corresponding log message updates. Remaining changes are formatting adjustments to .csproj files and Directory.Build.props.

Changes

Cohort / File(s)	Summary
Firewall logic update src/libs/H.Firewall/SessionExtensions.cs	Changed AddSubNetworkV4 action in BlockSubNetworkV4 from Permit to Block; updated log message from "Permit traffic on LAN network" to "Block traffic on network"
Project file formatting src/libs/Directory.Build.props, src/tests/H.Firewall.IntegrationTests/H.Firewall.IntegrationTests.csproj, src/tests/H.IpHlpApi.IntegrationTests/H.IpHlpApi.IntegrationTests.csproj, src/tests/H.OpenVpn.IntegrationTests/H.OpenVpn.IntegrationTests.csproj, src/tests/H.Wfp.IntegrationTests/H.Wfp.IntegrationTests.csproj	Whitespace and line-structure normalization; no functional or configuration changes

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Attention area: Verify that the semantic change in BlockSubNetworkV4 (from Permit to Block) is intentional and aligns with the method's intended behavior and caller expectations.

Poem

🐰 A firewall rabbit, quick and keen,
Now blocks the networks in between!
Where once packets passed with care,
Now none slip through—the net's so fair! ✨

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch claude/issue-45-vpn-011CUvq6sfe9jsREXgNE5vcs

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d856a3d and 1ae7d79.

📒 Files selected for processing (6)

• src/libs/Directory.Build.props (1 hunks)
• src/libs/H.Firewall/SessionExtensions.cs (2 hunks)
• src/tests/H.Firewall.IntegrationTests/H.Firewall.IntegrationTests.csproj (1 hunks)
• src/tests/H.IpHlpApi.IntegrationTests/H.IpHlpApi.IntegrationTests.csproj (1 hunks)
• src/tests/H.OpenVpn.IntegrationTests/H.OpenVpn.IntegrationTests.csproj (1 hunks)
• src/tests/H.Wfp.IntegrationTests/H.Wfp.IntegrationTests.csproj (1 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}