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

eyewitness: add page #9298

Closed
wants to merge 11 commits into from
Closed

eyewitness: add page #9298

wants to merge 11 commits into from

Conversation

ciph3rz
Copy link
Contributor

@ciph3rz ciph3rz commented Oct 23, 2022

  • [x ] The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • [ x] The page(s) have at most 8 examples.
  • [ x] The page description(s) have links to documentation or a homepage.
  • [ x] The page(s) follow the content guidelines.
  • [ x] The PR title conforms to the recommended templates.
  • Version of the command being documented (if known):

@github-actions github-actions bot added the new command Issues requesting creation of a new page. label Oct 23, 2022
@tldr-bot

This comment was marked as outdated.

@ciph3rz
Copy link
Contributor Author

ciph3rz commented Oct 23, 2022

Reference to #6863

pages/linux/eyewitness.md Outdated Show resolved Hide resolved

- Scan a single website using --single flag:

`eyewitness --single {{website}}`
Copy link
Member

Choose a reason for hiding this comment

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

The command has this case

Suggested change
`eyewitness --single {{website}}`
`EyeWitness --single {{website}}`

Copy link
Member

Choose a reason for hiding this comment

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

The command has this case

Yes it is mentioned here https://www.christophertruncer.com/eyewitness-2-0-release-and-user-guide/

Also, .py file extension is used

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Using the version from the Kali repos, it does not require specific capitalization nor the python extension. If we amend the TLDR to reflect this capitalization it will not run on the commandline.

Copy link
Member

Choose a reason for hiding this comment

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

in that case we should add an alias page.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just tried to add an alias command for it however TLDR bot does not like the capitalization of the command since I was using EyeWitness.

@kbdharun kbdharun changed the title eyewitness: initial commit eyewitness: add page Oct 24, 2022
pages/linux/eyewitness.md Outdated Show resolved Hide resolved

- Scan a single website using --single flag:

`eyewitness --single {{website}}`
Copy link
Member

Choose a reason for hiding this comment

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

The command has this case

Yes it is mentioned here https://www.christophertruncer.com/eyewitness-2-0-release-and-user-guide/

Also, .py file extension is used

pages/linux/eyewitness.md Outdated Show resolved Hide resolved
pages/linux/eyewitness.md Outdated Show resolved Hide resolved
ciph3rz and others added 5 commits October 28, 2022 17:25
Co-authored-by: K.B.Dharun Krishna <[email protected]>
Co-authored-by: K.B.Dharun Krishna <[email protected]>
Co-authored-by: K.B.Dharun Krishna <[email protected]>
@kbdharun kbdharun added the hacktoberfest-accepted PRs that were opened for Hacktoberfest, but may not actually get merged until November. label Oct 31, 2022
@tldr-bot
Copy link

The build for this PR failed with the following error(s):

pages/linux/EyeWitness.py.md:0: TLDR109 File name should be lowercase

Please fix the error(s) and push again.

@tldr-bot
Copy link

tldr-bot commented Nov 2, 2022

The build for this PR failed with the following error(s):

pages/linux/EyeWitness.py.md:0: TLDR109 File name should be lowercase

Please fix the error(s) and push again.

@tldr-bot
Copy link

tldr-bot commented Nov 2, 2022

The build for this PR failed with the following error(s):

pages/linux/EyeWitness.py.md:0: TLDR109 File name should be lowercase

Please fix the error(s) and push again.

Copy link
Member

@kbdharun kbdharun left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution. I have some suggestions for this page.


`eyewitness -f {{filename}} -d {{foldername}}`

- Scan muliple urls from xml file (nmap and nessus xml file format) and output to folername:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Scan muliple urls from xml file (nmap and nessus xml file format) and output to folername:
- Scan multiple URLs from XML file (`nmap` and `nessus` XML file format) and output to a specific directory:

@@ -0,0 +1,24 @@
# EyeWitness

> Python based tool to gather screenshots of websites, as well as additional information such as header info and default credentials if possible.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> Python based tool to gather screenshots of websites, as well as additional information such as header info and default credentials if possible.
> Python-based tool to gather screenshots of websites, as well as additional information such as header info and default credentials if possible.

> Python based tool to gather screenshots of websites, as well as additional information such as header info and default credentials if possible.
> More information: <https://www.christophertruncer.com/eyewitness-triage-tool>.

- Scan a single website using `--single` flag:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Scan a single website using `--single` flag:
- Scan a single website using the `--single` flag:


`EyeWitness --single {{website}}`

- Scan multiple urls from file and output to foldername:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Scan multiple urls from file and output to foldername:
- Scan multiple URLs from a file and output to a specific directory:


`eyewitness -f {{filename}} --useragent {{Mozilla/4.0}}`

- Scan multiple URLs from file using the `--jitter` flag to randomize accessing each URL with a base number of seconds:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Scan multiple URLs from file using the `--jitter` flag to randomize accessing each URL with a base number of seconds:
- Scan multiple URLs from a file using the `--jitter` flag to randomize accessing each URL with a base number of seconds:


- Scan multiple URLs from a file providing a specific user agent such as Mozilla/4.0:

`eyewitness -f {{filename}} --useragent {{Mozilla/4.0}}`
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`eyewitness -f {{filename}} --useragent {{Mozilla/4.0}}`
`eyewitness -f {{path/to/file}} --useragent {{Mozilla/4.0}}`


- Scan muliple urls from xml file (nmap and nessus xml file format) and output to folername:

`eyewitness -x {{filename}} -d {{foldername}}`
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`eyewitness -x {{filename}} -d {{foldername}}`
`eyewitness -x {{path/to/file.xml}} -d {{path/to/directory}}`


- Scan multiple urls from file and output to foldername:

`eyewitness -f {{filename}} -d {{foldername}}`
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`eyewitness -f {{filename}} -d {{foldername}}`
`eyewitness -f {{path/to/file}} -d {{path/to/directory}}`

Copy link
Contributor

Choose a reason for hiding this comment

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

Note for the PR author: it's a path placeholder format violation: mandatory fix.


- Scan a single website using `--single` flag:

`EyeWitness --single {{website}}`
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`EyeWitness --single {{website}}`
`EyeWitness --single {{website_url}}`

Would this be better?

# EyeWitness

> Python based tool to gather screenshots of websites, as well as additional information such as header info and default credentials if possible.
> More information: <https://www.christophertruncer.com/eyewitness-triage-tool>.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> More information: <https://www.christophertruncer.com/eyewitness-triage-tool>.
> More information: <https://github.com/FortyNorthSecurity/EyeWitness>.

I think it is better to link to their GitHub page as their usage guides seems to be updated frequently.

@github-actions
Copy link

Hi all! This thread has not had any recent activity.
Are there any updates? Thanks!

@github-actions github-actions bot added the waiting Issues/PRs with Pending response by the author. label Nov 25, 2022
Copy link
Contributor

@EmilyGraceSeville7cf EmilyGraceSeville7cf left a comment

Choose a reason for hiding this comment

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

  • No unmentioned must to fix issued found, approved.
  • This page should be reviewed by eyewitness users.
  • Thanks for a contribution! ❤️

> Python based tool to gather screenshots of websites, as well as additional information such as header info and default credentials if possible.
> More information: <https://www.christophertruncer.com/eyewitness-triage-tool>.

- Scan a single website using `--single` flag:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Scan a single website using `--single` flag:
- Scan a specific website:
  • Optional fix: a specific may sound better.
  • Optional fix: there is no need to explicitly mention flag names until it's unclear (definition is missing in our guides).


- Scan a single website using `--single` flag:

`EyeWitness --single {{website}}`
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
`EyeWitness --single {{website}}`
`EyeWitness --single "{{website_url}}"`

Option fix: possible missing quotes: quotes can be added if the placeholder can contain some special characters for POSIX-like shells.


- Scan multiple urls from file and output to foldername:

`eyewitness -f {{filename}} -d {{foldername}}`
Copy link
Contributor

Choose a reason for hiding this comment

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

Note for the PR author: it's a path placeholder format violation: mandatory fix.

@github-actions github-actions bot removed the waiting Issues/PRs with Pending response by the author. label Dec 9, 2022
@github-actions
Copy link

Hi all! This thread has not had any recent activity.
Are there any updates? Thanks!

@github-actions github-actions bot added the waiting Issues/PRs with Pending response by the author. label Dec 25, 2022
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions bot removed the waiting Issues/PRs with Pending response by the author. label Dec 29, 2022
@github-actions
Copy link

Hi all! This thread has not had any recent activity.
Are there any updates? Thanks!

@github-actions github-actions bot added the waiting Issues/PRs with Pending response by the author. label Apr 23, 2023
@github-actions
Copy link

Hi everyone.
This thread is being closed as there was no response to the previous prompt.
However, please leave a comment whenever you're ready to resume, so the thread can be reopened.
Thanks again!

@github-actions github-actions bot closed this May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted PRs that were opened for Hacktoberfest, but may not actually get merged until November. new command Issues requesting creation of a new page. waiting Issues/PRs with Pending response by the author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants