cURL / Mailing Lists / curl-library / Single Mail


Re: [PATCH] Support for Mozilla's Public Suffix List

From: Daniel Stenberg <>
Date: Fri, 9 Oct 2015 23:26:56 +0200 (CEST)

On Wed, 30 Sep 2015, Tim Ruehsen wrote:

Okay, back on track!

> I added test1136 (with PSL as keyword - not sure if it must be lower- or
> uppercase).

It looks like all the code assumes uppercase.

> It works when libpsl is compiled/linked, else it doesn't.
> I did a ' 1136', but did not have the time to run a 'make check' to
> see if the test is skipped when compiled without libpsl.

Two nits and a question.

1. try 'make checksrc' and it'll point out a few style nits (or use
./configure --enable-debug and then ordinary make invokes will do that for

2. there are fprintf() calls left in the code and commented out infof() calls

Q: This uses a built-in PSL database within libpsl, right? Is there any
support or plans for externally providing an updated PSL? I'm thinking perhaps
someone who uses libcurl with PSL in systems that have a life time expectensy
of some time may want to have means to update the db rather than to update the
lib? (it doesn't really concern this patch, its just a general question about

List admin:
Received on 2015-10-09