You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
socketserver accept a last request after shutdown #79198
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee=Noneclosed_at=<Date2018-10-26.15:15:59.308>created_at=<Date2018-10-18.11:55:13.111>labels= ['type-bug', 'library']
title='socketserver accept a last request after shutdown'updated_at=<Date2018-10-26.15:35:00.602>user='https://github.com/beledouxdenis'
After the shutdown of a BaseServer, the server may accept a last single request if it is sent between the server socket polling and the polling timeout.
This can be problematic for instance for a server restart for which you do not want to interrupt the service, by not closing the listening socket during the restart. One request can fail because of this behavior.
Note that only one request will fail, following requests will not be accepted, as expected.
New changeset 10cb376 by Victor Stinner (Denis Ledoux) in branch 'master': bpo-35017, socketserver: don't accept request after shutdown (GH-9952) 10cb376
New changeset 6f97a50 by Victor Stinner (Denis Ledoux) in branch '2.7': bpo-35017, socketserver: don't accept request after shutdown (GH-9952) (GH-10129) 6f97a50
Activity
beledouxdenis commentedon Oct 18, 2018
After the shutdown of a
BaseServer, the server may accept a last single request if it is sent between the server socket polling and the polling timeout.This can be problematic for instance for a server restart for which you do not want to interrupt the service, by not closing the listening socket during the restart. One request can fail because of this behavior.
Note that only one request will fail, following requests will not be accepted, as expected.
vstinner commentedon Oct 26, 2018
New changeset 10cb376 by Victor Stinner (Denis Ledoux) in branch 'master':
bpo-35017, socketserver: don't accept request after shutdown (GH-9952)
10cb376
miss-islington commentedon Oct 26, 2018
New changeset 9080824 by Miss Islington (bot) in branch '3.7':
bpo-35017, socketserver: don't accept request after shutdown (GH-9952)
9080824
miss-islington commentedon Oct 26, 2018
New changeset 8b1f52b by Miss Islington (bot) in branch '3.6':
bpo-35017, socketserver: don't accept request after shutdown (GH-9952)
8b1f52b
vstinner commentedon Oct 26, 2018
New changeset 6f97a50 by Victor Stinner (Denis Ledoux) in branch '2.7':
bpo-35017, socketserver: don't accept request after shutdown (GH-9952) (GH-10129)
6f97a50
vstinner commentedon Oct 26, 2018
Thanks Denis Ledoux for your bug report and your fix!
It's nice to see this bug fixed in all branches ;-)
beledouxdenis commentedon Oct 26, 2018
The pleasure is all mine.
Thanks to you and the other contributors involved. For my first contribution to Python, I am glad everything went smoothly :).