Skip to content

Added exception.py to setuptools script #101

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

Merged
merged 1 commit into from
Apr 15, 2020
Merged

Added exception.py to setuptools script #101

merged 1 commit into from
Apr 15, 2020

Conversation

ojasookert
Copy link

@ojasookert ojasookert commented Apr 15, 2020

After Added base exception class NoSQLMapException inside exception.py. #99, using NoSQLMap when installed with python setup.py install fails with the following log:

root@kali:~/NoSQLMap# nosqlmap.py --attack 2 --victim target-site.com --webPort 80 --uri /login --httpMethod POST --postData email,user@target-site.com,password,password --injectedParameter 1 --injectSize 0 --injectFormat 2 --savePath output.log
Traceback (most recent call last):
  File "/usr/local/bin/nosqlmap.py", line 4, in <module>
    __import__('pkg_resources').run_script('NoSQLMap==0.7', 'nosqlmap.py')
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 666, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 1469, in run_script
    exec(script_code, namespace, namespace)
  File "/usr/local/lib/python2.7/dist-packages/NoSQLMap-0.7-py2.7.egg/EGG-INFO/scripts/nosqlmap.py", line 6, in <module>
    
ImportError: No module named exception

Without installing with python setup.py install, this command completes successfully. After some debugging it looks like the exception.py is not installed with setuptools - adding the filename to the scripts list in setup.py fixes the issue.

@bbeale
Copy link
Contributor

bbeale commented Apr 15, 2020

Whoops, that one was me. I forgot to update setup.py before submitting the PR. Thanks for catching.

@codingo codingo merged commit 1b6552f into codingo:master Apr 15, 2020
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.

3 participants