Fix to reapply hostapd after wifi wan removed

[xinnige]

https://github.com/firewalla/firecommit/issues/7775

[j-sallyjin]

PR Review Summary

✅ What looks good

• Detects EVENT_WPA_DISCONNECTED specifically on the Wi-Fi client interface.
• Limits the behavior to APC-managed WLAN platforms.
• Debounces repeated disconnect events with a single delayed reapply timer.
• Uses the active config as the source of truth instead of constructing a partial hostapd-only change.

⚠️ Issues found

• Linked issue found: firewalla/firecommit#7775, but GitHub API access failed with HTTP Error 404, so issue requirement verification is limited to the PR title/body and provided diff.

• No blocking correctness, security, or networking issues found in the provided diff.

💡 Suggestions

• Consider adding a guard before reapply to confirm hostapd/AP config is actually present, since the helper name/log says “hostapd is configured” but the code currently reapplies on any matching WPA disconnect.
• Add regression coverage or a manual test note for: Wi-Fi WAN removal → WPA disconnect → hostapd/AP remains available after delayed reapply.
• Consider clearing _hostapdReapplyTimer during plugin teardown if this plugin has a lifecycle cleanup hook.

Verdict

APPROVE

---

Repo: firewalla/firerouter
PR: #1857
Head SHA: 52c2022b5b92bce872252058e552714428863ad7
Checked at: 2026-05-13 13:18:46 CST