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

Database imports #38

Open
d-Rickyy-b opened this issue Oct 7, 2018 · 1 comment
Open

Database imports #38

d-Rickyy-b opened this issue Oct 7, 2018 · 1 comment
Assignees
Labels
Difficulty: Hard This issue is hard to resolve priority:high An issue which has a high priority and needs a fix asap

Comments

@d-Rickyy-b
Copy link
Owner

  1. There are missing imports in the requirements.txt

  2. Currently when using pastepwn a user needs to download ALL the db connector packages (mongo & mysql currently). Depending on the amount of db connectors which at some point will be supported by pastepwn, it might be stupid to have them all in the same package.


Possible solution(s):

  1. try-except the error and return an error message. This is not elegant but allows everything to be bundeled in one package.
  2. move the database connectors to a different package. That way it's way cleaner but users need to install multiple packages.
@d-Rickyy-b d-Rickyy-b self-assigned this Oct 7, 2018
@d-Rickyy-b d-Rickyy-b added the priority:high An issue which has a high priority and needs a fix asap label Oct 8, 2018
@d-Rickyy-b d-Rickyy-b added the Difficulty: Hard This issue is hard to resolve label Oct 3, 2019
@d-Rickyy-b
Copy link
Owner Author

Optional packages could help fixing this issue.

https://stackoverflow.com/questions/11240285/can-i-define-optional-packages-in-setuptools

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Difficulty: Hard This issue is hard to resolve priority:high An issue which has a high priority and needs a fix asap
Projects
None yet
Development

No branches or pull requests

1 participant