Skip to content
Commit 3f697e30 authored by Dawit Alemayehu's avatar Dawit Alemayehu
Browse files

Make sure both DELETE and PUT requests properly handle redirection requests

to address the issue reported in bug# 267063. In the process of addressing the
afformentioned bug the following related changes and cleanups were performed:

  - Removed the check for HTTP_POST on redirection so that kio_http behaves
    exactly like Firefox and Chromium except for HEAD requests. Converting
    HEAD requests to GET makes no sense.

  - Changed both ::del and ::put to leave the query component of the request
    URL alone except for WebDAV requests. Not sure why it is removed for DAV
    requests either.

  - Changed ::del to properly handles any message body returned by the server.

  - Renamed sendHttpPutError to sendHttpError and converted it to handle HTTP
    response code errors for all HTTP methods, not just PUT.

  - Changed proceedUntilResponseContent to use the new sendHttpError function
    to send error to the client.

  - Converted both ::del and ::put to use proceedUntilResponseContent since it
    now correctly handles errors returned by the server.

BUG: 267063
FIXED-IN: 4.7.3
parent 1735f3f5
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment