Skip to content

Updates the services check#88

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.

Sign up for GitHub

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

Jump to bottom
Merged
Ekultek merged 3 commits into from
Mar 4, 2018
Merged

Updates the services check #88

Ekultek merged 3 commits into from
Mar 4, 2018

Conversation

@Ekultek
Copy link
Contributor

@EkultekEkultek commented Mar 2, 2018

There was an error that sometimes occurs after the service is started, fixed it:

[?] it appears that service apache2 is not enabled, would you like us to enable it for you[y/N]: y [+] services started successfully Traceback (most recent call last): File "autosploit.py", line 5, in <module> main() File "/home/baal/bin/python/autosploit/autosploit/main.py", line 33, in main while not check_services(service): File "/home/baal/bin/python/autosploit/lib/settings.py", line 93, in check_services running_proc = psutil.Process(pid) File "/home/baal/.local/lib/python2.7/site-packages/psutil/__init__.py", line 392, in __init__ self._init(pid) File "/home/baal/.local/lib/python2.7/site-packages/psutil/__init__.py", line 432, in _init raise NoSuchProcess(pid, None, msg) psutil.NoSuchProcess: psutil.NoSuchProcess no process found with pid 29144

Changed the color of misc_info so it's not as close to the prompt color:
new_colors

@EkultekEkultek requested a review from NullArrayMarch 2, 2018 17:13
thehappydinoa
thehappydinoa approved these changes Mar 2, 2018
View reviewed changes
Copy link
Contributor

@thehappydinoathehappydinoa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

autosploit/main.py
whilenotcheck_services(service):
choice=prompt("it appears that service{} is not enabled, would you like us to enable it for you[y/N]".format(service))
choice=prompt(
"it appears that service{} is not enabled, would you like us to enable it for you[y/N]".format(
Copy link
Contributor

@thehappydinoathehappydinoaMar 2, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a space after you ?

Copy link
ContributorAuthor

@EkultekEkultekMar 2, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

? you mean after the you[y/N] part?

@EkultekEkultek merged commit b5b0c9f into dev-betaMar 4, 2018
@EkultekEkultek deleted the service-edits branch March 4, 2018 01:59
@EkultekEkultek mentioned this pull request Apr 3, 2018
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NullArrayNullArrayAwaiting requested review from NullArray

1 more reviewer

@thehappydinoathehappydinoathehappydinoa approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Ekultek@thehappydinoa