diff options
author | Simon McVittie <smcv@debian.org> | 2019-02-10 17:22:06 +0000 |
---|---|---|
committer | Simon McVittie <smcv@debian.org> | 2019-02-26 22:21:27 +0000 |
commit | d283e4ca1aeb6ca8cc0951c8495f778071076013 (patch) | |
tree | c6e10b583199d2157242b4bc31b22fb2c4ac210d /IkiWiki/Plugin/blogspam.pm | |
parent | 67543ce1d62161fdef9dca198289d7dd7dceacc0 (diff) | |
download | ikiwiki-d283e4ca1aeb6ca8cc0951c8495f778071076013.tar ikiwiki-d283e4ca1aeb6ca8cc0951c8495f778071076013.tar.gz |
useragent: Automatically choose whether to use LWPx::ParanoidAgent
The simple implementation of this, which I'd prefer to use, would be:
if we can import LWPx::ParanoidAgent, use it; otherwise, use
LWP::UserAgent.
However, aggregate has historically worked with proxies, and
LWPx::ParanoidAgent quite reasonably refuses to work with proxies
(because it can't know whether those proxies are going to do the same
filtering that LWPx::ParanoidAgent would).
Signed-off-by: Simon McVittie <smcv@debian.org>
Diffstat (limited to 'IkiWiki/Plugin/blogspam.pm')
-rw-r--r-- | IkiWiki/Plugin/blogspam.pm | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/IkiWiki/Plugin/blogspam.pm b/IkiWiki/Plugin/blogspam.pm index 3eb4cf8b3..3835f52ca 100644 --- a/IkiWiki/Plugin/blogspam.pm +++ b/IkiWiki/Plugin/blogspam.pm @@ -57,18 +57,10 @@ sub checkconfig () { }; error $@ if $@; - eval q{use LWPx::ParanoidAgent}; - if (!$@) { - $client=LWPx::ParanoidAgent->new(agent => $config{useragent}); - } - else { - eval q{use LWP}; - if ($@) { - error $@; - return; - } - $client=useragent(); - } + # Using the for_url parameter makes sure we crash if used + # with an older IkiWiki.pm that didn't automatically try + # to use LWPx::ParanoidAgent. + $client=useragent(for_url => $config{blogspam_server}); } sub checkcontent (@) { |