This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author martin.panter
Recipients gmixo, martin.panter
Date 2015-12-11.10:26:09
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1449829569.73.0.0807659780231.issue25838@psf.upfronthosting.co.za>
In-reply-to
Content
Sorry but I have a few concerns about your patch:

Why does this have to be in the HTTPConnection.send() method? Why can’t you do the file rewinding yourself, when you retry the request?

This could break compatibility if someone wrote code that expects data to be sent from a non-zero file position.

I think this would be new feature (for the next version of Python), and couldn’t be accepted as a bug fix for 2.7.

See also Issue 9740 (persistent HTTP client connections), where I suggested a few things you mentioned, including polling for an unsolicited response or closed connection, and reconnecting before sending a request.

Also beware that a general HTTP client shouldn’t automatically retry idempotent requests if there is a chance that the original request was already received. PUT is idempotent so that is okay. POST is not, however.
History
Date User Action Args
2015-12-11 10:26:09martin.pantersetrecipients: + martin.panter, gmixo
2015-12-11 10:26:09martin.pantersetmessageid: <1449829569.73.0.0807659780231.issue25838@psf.upfronthosting.co.za>
2015-12-11 10:26:09martin.panterlinkissue25838 messages
2015-12-11 10:26:09martin.pantercreate