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

emacs-devel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From:	Dmitry Gutov
Subject:	Re: [PATCH] url: Wrap cookie headers in url-http--encode-string.
Date:	Sat, 10 Sep 2016 11:34:28 +0300
User-agent:	Mozilla/5.0 (X11; Linux x86_64; rv:49.0) Gecko/20100101 Thunderbird/49.0

On 10.09.2016 08:42, Eli Zaretskii wrote:

Well, according to this:

http://stackoverflow.com/a/1969339

Safari, at least, will reject non-ASCII cookies. Which implies that in
practice no sites will use non-ASCII values because they would break.


Then your original patch should be fine for the emacs-25 branch.  If
no objections are voiced here by the end of the day, I think we should
install it.


Agreed.

[Prev in Thread]

Current Thread

[Next in Thread]

Re: [PATCH] url: Wrap cookie headers in url-http--encode-string., (continued)
- Re: [PATCH] url: Wrap cookie headers in url-http--encode-string., Lars Ingebrigtsen, 2016/09/07
  - Re: [PATCH] url: Wrap cookie headers in url-http--encode-string., Toke Høiland-Jørgensen, 2016/09/07
  - Re: [PATCH] url: Wrap cookie headers in url-http--encode-string., Eli Zaretskii, 2016/09/07
    - Re: [PATCH] url: Wrap cookie headers in url-http--encode-string., Lars Ingebrigtsen, 2016/09/08

Prev by Date: Re: Using file descriptors in Emacs
Next by Date: Re: About color syntax in bash script
Previous by thread: Re: [PATCH] url: Wrap cookie headers in url-http--encode-string.
Next by thread: Re: [PATCH] url: Wrap cookie headers in url-http--encode-string.
Index(es):
- Date
- Thread