[issue26586] Simple enhancement to BaseHTTPRequestHandler

2016-04-02 Thread Martin Panter
Martin Panter added the comment: I didn’t touch 2.7 because that doesn’t have the same header parsing code as Python 3. -- resolution: -> fixed stage: commit review -> resolved status: open -> closed versions: -Python 2.7 ___ Python tracker

[issue26586] Simple enhancement to BaseHTTPRequestHandler

2016-04-02 Thread Roundup Robot
Roundup Robot added the comment: New changeset f5247195238f by Martin Panter in branch '3.5': Issue #26586: Handle excessive header fields in http.server, by Xiang Zhang https://hg.python.org/cpython/rev/f5247195238f New changeset e8edddb4f74b by Martin Panter in branch 'default': Issue #26586:

[issue26586] Simple enhancement to BaseHTTPRequestHandler

2016-04-02 Thread Martin Panter
Martin Panter added the comment: I propose to commit the “Too many headers” handler as a bug fix, and the other changes to only 3.6. Currently when more than 100 header fields are sent, the connection is killed and the server logs an unhandled exception. -- stage: patch review -> commi

[issue26586] Simple enhancement to BaseHTTPRequestHandler

2016-03-20 Thread Xiang Zhang
Xiang Zhang added the comment: Ping. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org

[issue26586] Simple enhancement to BaseHTTPRequestHandler

2016-03-19 Thread Xiang Zhang
Xiang Zhang added the comment: Thanks for your review. I have made some changes according to it. Status code 431 is returned when the header line is too long or there are too many headers. And I think the short message is OK and we can provide the exception message in the content message. As

[issue26586] Simple enhancement to BaseHTTPRequestHandler

2016-03-19 Thread Xiang Zhang
New submission from Xiang Zhang: This patch propose several simple enhancement to BaseHTTPRequestHandler. It removes some redundant code and add handler for too many headers. -- components: Library (Lib) files: simple_enhancement.patch keywords: patch messages: 261946 nosy: xiang.zhang

[issue26586] Simple enhancement to BaseHTTPRequestHandler

2016-03-18 Thread Martin Panter
Martin Panter added the comment: Thanks for the patch, it looks good in general. I left some comments and suggestions. -- nosy: +martin.panter stage: -> patch review ___ Python tracker ___