search

RedSiege / EyeWitness

EyeWitness is designed to take screenshots of websites, provide some server header info, and identify default credentials if possible.
https://www.christophertruncer.com/eyewitness-usage-guide/
GNU General Public License v3.0
5.02k stars 851 forks source link

Prevent terminal screen buffer from being cleared when running application #552

Closed denevin closed 6 months ago

denevin commented 2 years ago

OS Used - ALL Information (architecture, linux flavor, etc.)

Linux

Expected behavior (vs. what you encountered)

When running eyewitness, the terminal screen buffer is cleared and all the information that was there previously is lost. During pentesting, previous information that was in the terminal screen buffer before might have not been saved yet. It would be great if the eyewitness application does not clear the terminal screen buffer when it's started and instead keeps information that was previously there so that it's possible to scroll up in the terminal screen to retried information from other applications that were ran before eyewitness.

n00py commented 2 years ago

456 also.

n00py commented 2 years ago

proposed fix: #559

sneakerhax commented 2 years ago

I would also like an argument added to avoid clearing the screen upon execution. When running this with automation, especially inside a container, the earlier stages of the automation output are cleared.

Relkci commented 6 months ago

@ChrisTruncer -- running #647 past you for your opinion. Rather than always clearing like in #559 , this MR passes an optional argument, with the default being a screen clear to your preference

ChrisTruncer commented 6 months ago

Thank you for highlighting this, and apologies for the business as of late. For what its worth, the proposed addition for not clearing the screen by having a CLI flag looks solid. I'm good with that.

0x6d6f7468 commented 6 months ago

Maybe this is me being dumb, but I've always wondered what the point of clearing the buffer automatically was.

For what it's worth, I would probably prefer not clearing to be default, but at that point I wonder if it would be better to just remove the feature altogether and leave that up to the user via the clear command, rather than adding another command line argument. Just my $0.02

Edit: I see that a PR was previously closed with just this suggestion, sounds like you prefer it to clear automatically. If that's still the case, then I won't bring it up again.

ChrisTruncer commented 6 months ago

When this was first being developed, I preferred it to do that, and still largely do, so that's why. But I don't mind the inclusion of an easy to add cli parameter that changes that

Relkci commented 6 months ago

Merged, closing