mirror of https://github.com/scrapy/scrapy.git
duplicatefilter: dont raise ignorerequest, but add request to filter on spider input
--HG-- extra : convert_revision : svn%3Ab85faa78-f9eb-468e-a121-7cced6da292c%40885
This commit is contained in:
parent
fe7935005c
commit
756cda3873
|
|
@ -39,8 +39,7 @@ class DuplicatesFilterMiddleware(object):
|
|||
dispatcher.connect(self.filter.close, signals.domain_closed)
|
||||
|
||||
def process_spider_input(self, response, spider):
|
||||
if not self.filter.add(spider.domain_name, response.request):
|
||||
raise IgnoreRequest("Skipped (already processed): %s" % response.request)
|
||||
self.filter.add(spider.domain_name, response.request)
|
||||
|
||||
def process_spider_output(self, response, result, spider):
|
||||
domain = spider.domain_name
|
||||
|
|
|
|||
|
|
@ -32,10 +32,6 @@ class DuplicatesFilterMiddlewareTest(unittest.TestCase):
|
|||
assert r2 in filtered
|
||||
assert r3 not in filtered
|
||||
|
||||
response = Response('http://scrapytest.org/')
|
||||
response.request = Request('http://scrapytest.org/')
|
||||
self.assertRaises(IgnoreRequest, mw.process_spider_input, response, self.spider)
|
||||
|
||||
mw.filter.close('scrapytest.org')
|
||||
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue