Skip to content
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

Web interface can crash when link-checker module is enabled #410

Closed
pajlada opened this issue Jul 7, 2019 · 1 comment
Closed

Web interface can crash when link-checker module is enabled #410

pajlada opened this issue Jul 7, 2019 · 1 comment
Labels

Comments

@pajlada
Copy link
Member

pajlada commented Jul 7, 2019

Callstack:

Jul 07 16:22:36 dune uwsgi[3158]: [2019-07-07 16:22:36,775] [ERROR  ] Unhandled exception
Jul 07 16:22:36 dune uwsgi[3158]: Traceback (most recent call last):
Jul 07 16:22:36 dune uwsgi[3158]:   File "/srv/pajbot/myvenv/lib/python3.5/site-packages/flask/app.py", line 1813, in full_dispatch_request
Jul 07 16:22:36 dune uwsgi[3158]:     rv = self.dispatch_request()
Jul 07 16:22:36 dune uwsgi[3158]:   File "/srv/pajbot/myvenv/lib/python3.5/site-packages/flask/app.py", line 1799, in dispatch_request
Jul 07 16:22:36 dune uwsgi[3158]:     return self.view_functions[rule.endpoint](**req.view_args)
Jul 07 16:22:36 dune uwsgi[3158]:   File "./pajbot/web/routes/base/commands.py", line 46, in command_detailed
Jul 07 16:22:36 dune uwsgi[3158]:     bot_commands_list = get_commands_list()
Jul 07 16:22:36 dune uwsgi[3158]:   File "./pajbot/web/routes/base/commands.py", line 9, in get_commands_list
Jul 07 16:22:36 dune uwsgi[3158]:     return pajbot.web.utils.get_cached_commands()
Jul 07 16:22:36 dune uwsgi[3158]:   File "./pajbot/utils.py", line 73, in wrap
Jul 07 16:22:36 dune uwsgi[3158]:     ret = f(*args)
Jul 07 16:22:36 dune uwsgi[3158]:   File "./pajbot/web/utils.py", line 106, in get_cached_commands
Jul 07 16:22:36 dune uwsgi[3158]:     socket_manager=None, module_manager=ModuleManager(None).load(), bot=None
Jul 07 16:22:36 dune uwsgi[3158]:   File "./pajbot/models/module.py", line 97, in load
Jul 07 16:22:36 dune uwsgi[3158]:     self.all_modules = [module(self.bot) for module in available_modules]
Jul 07 16:22:36 dune uwsgi[3158]:   File "./pajbot/models/module.py", line 97, in <listcomp>
Jul 07 16:22:36 dune uwsgi[3158]:     self.all_modules = [module(self.bot) for module in available_modules]
Jul 07 16:22:36 dune uwsgi[3158]:   File "./pajbot/modules/followage.py", line 60, in __init__
Jul 07 16:22:36 dune uwsgi[3158]:     self.action_queue.start()
Jul 07 16:22:36 dune uwsgi[3158]:   File "./pajbot/actions.py", line 35, in start
Jul 07 16:22:36 dune uwsgi[3158]:     t.start()
Jul 07 16:22:36 dune uwsgi[3158]:   File "/usr/lib/python3.5/threading.py", line 844, in start
Jul 07 16:22:36 dune uwsgi[3158]:     _start_new_thread(self._bootstrap, ())
Jul 07 16:22:36 dune uwsgi[3158]: RuntimeError: can't start new thread

The action_queue should probably be started in enable or something instead? or maybe just a lazy load when the first link is checked

@RAnders00
Copy link
Contributor

I do believe this is fixed since initializing the linkchecker no longer creates an ActionQueue (since 94de6f2#diff-b79c3a3af53423e1d2a4a1e05e844046 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants