[PR #162] [MERGED] Gracefully shutdown uvicorn.Server on task cancellation in run() #180

New Issue

2026-02-15T21:15:14-05:00

yindo commented

2026-02-15 21:15:14 -05:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/mcpo/pull/162
Author: @021gink
Created: 5/28/2025
Status: ✅ Merged
Merged: 6/6/2025
Merged by: @tjbck

Base: main ← Head: graceful-uvicorn-shutdown

📝 Commits (1)

101c513 Gracefully shutdown uvicorn.Server on task cancellation in run()

📊 Changes

1 file changed (+7 additions, -1 deletions)

View changed files

📝 src/mcpo/main.py (+7 -1)

📄 Description

Summary

This PR updates the run() function to properly shut down the uvicorn.Server when the task is cancelled via task.cancel().

Motivation

Previously, cancelling the task running run() did not stop the uvicorn server, leaving the socket bound to the port and causing issues when trying to restart or cleanly terminate the server. This change handles asyncio.CancelledError and triggers server.should_exit = True and await server.shutdown() to release resources.

Changes

Added try/except asyncio.CancelledError block around server.serve()
On cancellation, server.should_exit is set and server.shutdown() is awaited before re-raising the cancellation

Testing

Tested locally by launching run() via asyncio.create_task() and then cancelling it. Confirmed that the uvicorn server exits cleanly and releases the port.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/mcpo/pull/162 **Author:** [@021gink](https://github.com/021gink) **Created:** 5/28/2025 **Status:** ✅ Merged **Merged:** 6/6/2025 **Merged by:** [@tjbck](https://github.com/tjbck) **Base:** `main` ← **Head:** `graceful-uvicorn-shutdown` --- ### 📝 Commits (1) - [`101c513`](https://github.com/open-webui/mcpo/commit/101c513aebf2744225a17d51deef27f2a2d98ea4) Gracefully shutdown uvicorn.Server on task cancellation in run() ### 📊 Changes **1 file changed** (+7 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/mcpo/main.py` (+7 -1) </details> ### 📄 Description ### Summary This PR updates the `run()` function to properly shut down the `uvicorn.Server` when the task is cancelled via `task.cancel()`. ### Motivation Previously, cancelling the task running `run()` did not stop the `uvicorn` server, leaving the socket bound to the port and causing issues when trying to restart or cleanly terminate the server. This change handles `asyncio.CancelledError` and triggers `server.should_exit = True` and `await server.shutdown()` to release resources. ### Changes - Added `try/except asyncio.CancelledError` block around `server.serve()` - On cancellation, `server.should_exit` is set and `server.shutdown()` is awaited before re-raising the cancellation ### Testing Tested locally by launching `run()` via `asyncio.create_task()` and then cancelling it. Confirmed that the uvicorn server exits cleanly and releases the port. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

yindo added the pull-request label 2026-02-15 21:15:14 -05:00

yindo closed this issue

2026-02-15 21:15:14 -05:00

yindo referenced this issue

2026-02-15 21:15:15 -05:00

[PR #180] [CLOSED] fix(server_type): SSE server does not reconnect if session lost #188

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: open-webui/mcpo#180