adjust shutdown process on the httpserver to remove fsm noise

[robbyt]

Previously, the httpserver.Runner shutdown process had two concurrent paths that could trigger FSM state transitions: explicit Stop() calls and context cancellation. This could cause race conditions where both paths attempted to transition the FSM from Running to Stopping state simultaneously, resulting in "current state mismatch" warnings.

This fixes that problem by consolidating all FSM transitions into a single shutdown() method. The Stop() method now only cancels the context, while the Run() method detects cancellation and calls shutdown() to handle state transitions. This eliminates the race condition by ensuring only one code path manages FSM state changes during shutdown.

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
89.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud