Git-Mirrors/brozzler
1
0
Fork 0
mirror of https://github.com/internetarchive/brozzler.git synced 2025-06-21 05:14:22 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

bugfix for BrozzlerWorker._needs_browsing

Browse source 
I'm sorry but with my previous commit I introduced a bug in
``BrozzlerWorker._needs_browsing`` method.

More specifically, if the ``brozzler_spy`` param is False (this happens
when ``youtube_dl`` is disabled), ``_needs_browsing`` method returns
always ``False`` and this messes with ``brozzle_page`` workflow. The
browser is nevenr visiting the page.

I'm fixing this here.
This commit is contained in:
Vangelis Banos 2017-08-01 11:43:42 +00:00
parent 895bfbf913
commit ae73edb244
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
brozzler/worker.py
View file

@ -447,7 +447,9 @@ class BrozzlerWorker:
if txn['response_headers'].get_content_type() in [ if txn['response_headers'].get_content_type() in [
'text/html', 'application/xhtml+xml']: 'text/html', 'application/xhtml+xml']:
return True return True
return False return False
else:
return True
def _already_fetched(self, page, brozzler_spy): def _already_fetched(self, page, brozzler_spy):
if brozzler_spy: if brozzler_spy: