[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-wget] [Bug-Wget] Misc. patches
From: |
Tim Ruehsen |
Subject: |
Re: [Bug-wget] [Bug-Wget] Misc. patches |
Date: |
Mon, 07 Jul 2014 16:44:22 +0200 |
User-agent: |
KMail/4.12.4 (Linux/3.14-1-amd64; KDE/4.13.1; x86_64; ; ) |
One more comment / idea.
The 'cookie_domain' comes from a HTTP Set-Cookie repsonse header and thus is
(must be) toASCII() encoded (=puncode). Of course this has to be checked when
normalizing the incoming cookie data. A cookie comain having non-ascii
characters should simply be dropped.
The whole check only works when 'host' is also in toASCII() (punycode) form.
Assuming this, psl_str_to_utf8lower() just reduces to a ASCII lowercase
converter.
If Wget would convert any domain name input to punycode + lowercase, many
conversions would fall away and case-function would not be needed (e.g.
calling strcmp instead of strcasecmp, the need to call psl_str_to_utf8lower()
would fall away, etc.).
What do you think ?
Tim
On Monday 07 July 2014 17:08:48 Darshit Shah wrote:
> + if (psl_str_to_utf8lower (cookie_domain, NULL, NULL,&cookie_domain_lower)
== PSL_SUCCESS &&
> + psl_str_to_utf8lower (host, NULL, NULL, &host_lower) == PSL_SUCCESS)
> + {
> + is_acceptable = psl_is_cookie_domain_acceptable (psl,
> host_lower, cookie_domain_lower);
> + }
> + else
> + {
> + DEBUGP (("libpsl unable to parse domain name. "
> + "Falling back to simple heuristics.\n"));
> + goto no_psl;
> + }
- [Bug-wget] [Bug-Wget] Misc. patches, Darshit Shah, 2014/07/05
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Giuseppe Scrivano, 2014/07/05
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Darshit Shah, 2014/07/05
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Giuseppe Scrivano, 2014/07/05
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Darshit Shah, 2014/07/05
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Tim Ruehsen, 2014/07/07
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Darshit Shah, 2014/07/07
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Tim Ruehsen, 2014/07/07
- Re: [Bug-wget] [Bug-Wget] Misc. patches,
Tim Ruehsen <=
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Darshit Shah, 2014/07/20
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Tim Rühsen, 2014/07/20
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Darshit Shah, 2014/07/21
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Giuseppe Scrivano, 2014/07/21
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Tim Rühsen, 2014/07/22
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Darshit Shah, 2014/07/23
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Darshit Shah, 2014/07/19
- Re: [Bug-wget] [Bug-Wget] Misc. patches, Tim Rühsen, 2014/07/19