Skip to content

XER10-3108 : Restart SSH after lease change.#218

Merged
guto86 merged 1 commit into
mainfrom
bugfix/XER10-3108-ssh-restart-2
Jul 13, 2026
Merged

XER10-3108 : Restart SSH after lease change.#218
guto86 merged 1 commit into
mainfrom
bugfix/XER10-3108-ssh-restart-2

Conversation

@S-Parthiban-Selvaraj

Copy link
Copy Markdown
Contributor

Restart WAN management services on IPv4 and IPv6 setup to ensure proper binding of management services to the new WAN addresses.

Restart WAN management services on IPv4 and IPv6 setup to ensure proper binding of management services to the new WAN addresses.
Copilot AI review requested due to automatic review settings July 13, 2026 12:45
@S-Parthiban-Selvaraj S-Parthiban-Selvaraj requested a review from a team as a code owner July 13, 2026 12:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates WAN interface bring-up flows to restart WAN management services after IPv4/IPv6 configuration so services (e.g., sshd / reverse SSH) rebind to newly acquired WAN addresses, including during lease/address changes.

Changes:

  • Moves the wanmgr_services_restart() call in the IPv4 setup path to occur after firewall restart triggering, and makes it run on both first bring-up and renew/change.
  • Moves the wanmgr_services_restart() call in the IPv6 setup path to occur after IPv6 setup/MAC sysevent handling, and makes it run on both first bring-up and renew/change.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/WanManager/wanmgr_interface_sm.c
Comment thread source/WanManager/wanmgr_interface_sm.c
@guto86 guto86 merged commit 2dc63db into main Jul 13, 2026
13 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants