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

Fixed : Improved downloading of requirements.txt #46

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kunalsz
Copy link

@kunalsz kunalsz commented Feb 3, 2025

in reference to the issue #45

This PR updates the honeyscanner/honeypots/base_honeypot.py

Changes made:

  • The versions and requirements.txt were hard coded, not covering latest versions released which is not scalable enough
  • A new function get_requirements is added to the BaseHoneypot class which retrieves the latest version and requirements.txt file of a honeypot
  • cowrie.py and conpot.py were updated as they have requirements.txt on their GitHub repo.

The error has been fixed and the code has been made more scalable

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

Successfully merging this pull request may close these issues.

1 participant