[Bug 59573] Regression introduced by decompression support in winhttp

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=59573

--- Comment #2 from mata <[e-mail]> ---
Actually, testing some more, it seems that the right header returned for
compressed content is for example

    Content-Encoding: deflate

But in this case, there wasn't even a copression related 'Accept: ' header set
in the first place. In the absence of that, assuming it has to be one of gzip
or deflate and otherwise setting

    else ret = ERROR_WINHTTP_INVALID_SERVER_RESPONSE;

seems wrong to me.
I'll also attach the logs I creatd from Overwatch.

I used a simple script for some tests following some microsoft examples:

    function testReq(url) {
        try {
            WScript.Echo("--- trying: " + url + " ---")
            var req = new ActiveXObject("WinHttp.WinHttpRequest.5.1");
            req.Open("GET", url , true);
            req.Send();
            req.WaitForResponse();
            WScript.Echo("--- headers ---")
            WScript.Echo(req.GetAllResponseHeaders())
            WScript.Echo("--- response text ---")
            WScript.Echo(req.ResponseText)
            WScript.Echo("--- response body ---")
            WScript.Echo(req.ResponseBody)
        } catch (err) {
            WScript.Echo(err + " - " + (err.number & 0xFFFF).toString())
        }
    }

    testReq("https://httpbin.org/deflate")
    testReq("https://httpbin.org/gzip")


That works with the original WINHTTP_QUERY_CONTENT_ENCODING.
gzip works, but the `deflate` case doesn't work unless i change inflateInit2()
window size argument from -15 to 15.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.