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

v2.1.2 default installation after instruction lacks dependencies for parsers/html_table #1499

Open
bernhardreiter opened this issue Jan 31, 2020 · 7 comments
Labels
bug Indicates an unexpected problem or unintended behavior documentation Indicates a need for improvements or additions to documentation help wanted Indicates that a maintainer wants help on an issue or pull request usability

Comments

@bernhardreiter
Copy link
Contributor

bernhardreiter commented Jan 31, 2020

Following v2.1.2 Install and Development instructions, then starting the example default bot net,
the feodo-tracker-browse-parser stopped with

  File "/opt/dev_intelmq/intelmq/bots/parsers/html_table/parser.py", line 37, in init
    raise MissingDependencyError("beautifulsoup4")
intelmq.lib.exceptions.MissingDependencyError: Could not load dependency 'beautifulsoup4', please install it with apt/yum/dnf/zypper (possibly named python3-beautifulsoup4) or pip3.

Expectation: the instructions or the setup process should make sure that the dependencies for the standard bots are there.

Workaround on Ubuntu 18.04 LTS:

apt-install python-bs4 python3-lxml

Idea: the intelmqctl check command should also check if dependencies can be loaded. Probably needs changes in the bots implementation.

@ghost ghost added the usability label Feb 3, 2020
@ghost ghost self-assigned this Feb 3, 2020
@ghost ghost closed this as completed in 6f71d74 Feb 3, 2020
@ghost ghost reopened this Feb 3, 2020
@ghost
Copy link

ghost commented Feb 3, 2020

Sorry for the accidental close, I mixed up the issue numbers -.-

@ghost ghost added the documentation Indicates a need for improvements or additions to documentation label Jul 15, 2020
@ghost ghost added this to the 2.2.1 milestone Jul 15, 2020
@ghost ghost added the bug Indicates an unexpected problem or unintended behavior label Jul 15, 2020
@ghost ghost modified the milestones: 2.2.1, 3.0.0 Jul 23, 2020
@ghost
Copy link

ghost commented Jul 23, 2020

See also #552

Requires some changes in the core. The requirements of bots need to be defined somewhere

@ghost ghost removed their assignment Feb 19, 2021
@ghost ghost removed this from the 3.0.0 milestone Jun 17, 2021
@motok
Copy link

motok commented Jan 18, 2022

intelmq 3.0.1 seems still have this issue.

$ intelmqctl -v
3.0.1
2022-01-18 16:30:35,475 - feodo-tracker-browse-parser - INFO - HTMLTableParserBot initialized with id feodo-tracker-browse-parser and intelmq 3.0.1 and python 3.8.12 (default, Nov  4 2021, 01:12:24) as process 769.
2022-01-18 16:30:35,475 - feodo-tracker-browse-parser - INFO - Bot is starting.
2022-01-18 16:30:35,650 - feodo-tracker-browse-parser - ERROR - Bot initialization failed.
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/intelmq/lib/bot.py", line 209, in __init__
    self.init()
  File "/usr/local/lib/python3.8/site-packages/intelmq/bots/parsers/html_table/parser.py", line 55, in init
    raise MissingDependencyError("beautifulsoup4")
intelmq.lib.exceptions.MissingDependencyError: Could not load dependency 'beautifulsoup4', please install it with apt/yum/dnf/zypper (possibly named python3-beautifulsoup4) or pip3.
2022-01-18 16:30:35,653 - feodo-tracker-browse-parser - INFO - Bot stopped.

@sebix
Copy link
Member

sebix commented Jan 18, 2022

Yes, it's the case for all bots which have specific additional requirements.

@sebix sebix added the help wanted Indicates that a maintainer wants help on an issue or pull request label Feb 3, 2022
@waldbauer-certat
Copy link
Contributor

This doesnt seems like a bug to me, because the error message should be clear enough. Maybe add some better error handling in intelmqctl and intelmq-manager if the error is MissingDependencyError.

Would remove the bug label, as its not a bug. @sebix

@sebix
Copy link
Member

sebix commented Jul 15, 2022

It's definitely bad usability. IIRC the reason I labelled this as bug was that this particular bot is part of the standard botnet. In other words: The default installation does not work without installing further dependencies. If you disagree, feel free to remove the label.

feodo-tracker-browse-parser:
description: HTML Table Parser is a bot configurable to parse different html table
data.
enabled: true
group: Parser
module: intelmq.bots.parsers.html_table.parser

kamil-certat added a commit to kamil-certat/intelmq that referenced this issue Jan 9, 2023
Feodotracker bots requires additional, heavy dependencies
(beautifulsoup4) to start. Disabling them in the default
conf allows us to ship the working config with the package
without increase the dependencies required to install
at the beginning.

Fix: certtools#1499
@kamil-certat
Copy link
Contributor

I suggest disabling those bots in the default config. The bs4 is a heavy dependency, and I don't think it makes sense to ship it for everyone. We may find other alternative bots to present the duplicator usage, but so far, shipping not working config by default doesn't make any sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior documentation Indicates a need for improvements or additions to documentation help wanted Indicates that a maintainer wants help on an issue or pull request usability
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants