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

bug(screenshot): screenshot scan fails with errors #110

Closed
3 tasks done
zinwelzl opened this issue Jun 16, 2024 · 6 comments · Fixed by #114
Closed
3 tasks done

bug(screenshot): screenshot scan fails with errors #110

zinwelzl opened this issue Jun 16, 2024 · 6 comments · Fixed by #114
Labels
bug Something isn't working

Comments

@zinwelzl
Copy link

Current Behavior

Scan screenshot fail, EyeWitness TERM environment variable not set

screenshot fail with this log.

Logs for scan

python3 /usr/src/github/EyeWitness/Python/EyeWitness.py -f /usr/src/scan_results/ginandjuice.shop/scans/81defeee-2bbd-11ef-91bc-0242ac130004/endpoints_alive.txt -d /usr/src/scan_results/ginandjuice.shop/scans/81defeee-2bbd-11ef-91bc-0242ac130004/screenshots --no-prompt --timeout 10 --threads 40

TERM environment variable not set.
################################################################################

EyeWitness

################################################################################

Red Siege Information Security - https://www.redsiege.com

################################################################################

Starting Web Requests (1 Hosts)
Attempting to screenshot https://ginandjuice.shop
Finished in 34.07110023498535 seconds

Expected Behavior

.

Steps To Reproduce

.

Environment

eNgine: https://github.com/Security-Tools-Alliance/rengine-ng
OS: WattOS 13
Browser: Firefox

Anything else?

No response

Acknowledgements

  • This issue is not a duplicate of an existing bug report.
  • I have chosen an appropriate title.
  • All requested information has been provided properly.
@zinwelzl zinwelzl added the bug Something isn't working label Jun 16, 2024
@psyray
Copy link
Contributor

psyray commented Jun 17, 2024

Can I have the content of the files in the command line ?

@zinwelzl
Copy link
Author

I deleted this version and VM and will install v2.1.0 this weekend.
Sorry.

@psyray
Copy link
Contributor

psyray commented Jun 18, 2024

I deleted this version and VM and will install v2.1.0 this weekend. Sorry.

Tried to reproduce with no success.
Did you try anothet target ?
Waiting for your feedback on the new setup

@psyray psyray changed the title Scan screenshot fail, EyeWitness TERM environment variable not set Scan screenshot fail Jun 19, 2024
@psyray psyray changed the title Scan screenshot fail bug(screenshot): screenshot scan fails with errors Jun 19, 2024
@psyray
Copy link
Contributor

psyray commented Jun 19, 2024

After more tests, I'm able to reproduce the error
It appears that EyeWitness doesn't work anymore, maybe due to the recent EyeWitness modification on bad URL
#15
#98

Current error is

celery-1       | run_command                        | INFO | python3 /usr/src/github/EyeWitness/Python/EyeWitness.py -f /usr/src/scan_results/domain.com/scans/a5c50400-2e17-11ef-9f19-0242ac170004/endpoints_alive.txt -d /usr/src/scan_results/domain.com/scans/a5c50400-2e17-11ef-9f19-0242ac170004/screenshots --no-prompt --timeout 10 --threads 40
celery-1       | screenshot                         | ERROR | too many values to unpack (expected 6)
celery-1       | Traceback (most recent call last):
celery-1       |   File "/usr/src/app/reNgine/celery_custom_task.py", line 129, in __call__
celery-1       |     self.result = self.run(*args, **kwargs)
celery-1       |   File "/usr/src/app/reNgine/tasks.py", line 1196, in screenshot
celery-1       |     protocol, port, subdomain_name, status, screenshot_path, source_path = tuple(row)
celery-1       | ValueError: too many values to unpack (expected 6)
celery-1       | screenshot                         | WARNING | Task screenshot status is FAILED | Error: ValueError('too many values to unpack (expected 6)')

It seems the format returned has changed and need to be fixed ASAP

@zinwelzl
Copy link
Author

I also get
#########################################################################
Installing Docker...

Executing docker install script, commit: 6d9743e9656cc56f699a64800b098d5ea5a60020

@psyray
Copy link
Contributor

psyray commented Jun 20, 2024

@zinwelzl
Fixed
Could you test by doing a git checkout of the https://github.com/Security-Tools-Alliance/rengine-ng/tree/fix-110-screenshot-error branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants