Re: [PATCH] url: Wrap cookie headers in url-http--encode-string.

[Toke Høiland-Jørgensen]

Alain Schneble <address@hidden> writes:

> Toke Høiland-Jørgensen <address@hidden> writes:
>
>> Dmitry Gutov <address@hidden> writes:
>>>
>>> Could you post the full recipe?
>>
>> (url-retrieve-synchronously "http://google.se";) ; sets a cookie
>> (let* ((url-request-data (encode-coding-string "æøå" 'utf-8)))
>>        (url-retrieve-synchronously "http://google.se";)) ; crashes
>
> Just two notes on this:
>
> 1. This example uses a GET request with some body data.  I don't think
>    that this is a real use case.  Of course, with other requests such as
>    POST, where some body data is typically sent, we might run into the
>    same issue.

Yeah, that was not supposed to be a real-world use case, just the
shortest code snippet I could get to crash. The actual request that I
ran into this bug with was a PUT.

-Toke