New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add accidentally removed import to search.py #1497

Merged
merged 2 commits into from Jan 29, 2019

Conversation

Projects
None yet
2 participants
@kvch
Copy link
Collaborator

kvch commented Jan 25, 2019

When reverting "remove 'all' option from search languages" (#1475) was merged, an import got removed accidentally. This leads to the following error in the console:

Exception in thread 419709b5-6c9a-472f-9337-55b5520e4c7a:
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
    self.run()
  File "/usr/lib/python2.7/threading.py", line 754, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/home/n/p/searx/searx/search.py", line 150, in search_one_request_safe
    engine.suspend_end_time = time() + min(settings['search']['max_ban_time_on_fail'],
NameError: global name 'settings' is not defined

This PR adds the import again.
@Pofilo if you have time, please review the PR.

@kvch kvch added bug core labels Jan 25, 2019

@Pofilo

Pofilo approved these changes Jan 27, 2019

Copy link
Collaborator

Pofilo left a comment

Perfect !

@kvch kvch merged commit 5fa3c11 into asciimoo:master Jan 29, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment