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

Small screen width leads to repeating lines in output #1038

Closed
watskenet opened this issue Nov 29, 2023 · 7 comments
Closed

Small screen width leads to repeating lines in output #1038

watskenet opened this issue Nov 29, 2023 · 7 comments
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@watskenet
Copy link

if my screen size small , show to me wrong data in screen

@watskenet watskenet added the bug Something isn't working label Nov 29, 2023
@Pharisee
Copy link

Pharisee commented Dec 8, 2023

Instead of opening a new issue, I'll just expand on this one since (if it's not just a troll) there's a chance it's related.

Essentially when the window is small enough that the length of one of the lines exceeds it, the output bugs out and starts repeating lines and scrolling down the screen endlessly in a real ugly way. My guess is it seems like a problem with however the output lines are being updated/refreshed that doesn't like when the lines go past the far edge of the terminal window and are forced to wrap.

Dunno rust shit, so no clue where the issue is or if it's fixable, but figured I'd drop it in here since it was bugging me trying to figure out why the output looked like trash when everybody was saying it looked dope until I figured out I just needed to widen the window. Anyways, it's quality shit dude. Appreciate ya.

image

@epi052 epi052 added the wontfix This will not be worked on label Dec 9, 2023
@epi052
Copy link
Owner

epi052 commented Dec 9, 2023

tysm! glad you're enjoying it

as for the output, this has been a known issue for a very long time. I use a library for handling terminal output and don't plan on trying to apply a fix for this particular issue. The workarounds are documented at the link below:

https://epi052.github.io/feroxbuster-docs/docs/faq/progress-bars/

i think the only reasonable fix is to check terminal width at startup and emit a warning to the user 🤔

@epi052 epi052 added the maybe label Dec 9, 2023
@epi052
Copy link
Owner

epi052 commented Feb 24, 2024

morning! i was going to add a warning on small terminal width, but i can't actually reproduce this anymore. could you give it a shot and see if you can still get it to do this @Pharisee

@Pharisee
Copy link

Ah. Sorry, I didn't respond before. Been a busy year so far. Just tested it again and it seems like the bug is only triggering in t-pain mode (aka --auto-tune) when the stoplight message displays (set rate limit, increased scan speed, etc.)

@epi052
Copy link
Owner

epi052 commented Feb 25, 2024

thanks for that! What terminal are you using? i've tried 3 separate terminal emulators and still can't repro, even with auto-tune.

all i really need to know is the terminal width at which i should warn users that things may get wonky. i could just say 80 or less and call it a day, but would like to be more specific. any chance you could get that info @Pharisee ?

@Pharisee
Copy link

Hmm, I wonder what's diifferent. I've been able to replicate it consistently with qTerminal, kitty, and alacritty. It's hard to say a specific width since it's just happening any time one of the directory listing lines at the bottom reaches the width of the terminal.

@epi052 epi052 changed the title [BUG] faild screen size Small screen width leads to repeating lines in output Feb 26, 2024
@epi052 epi052 removed the maybe label Feb 26, 2024
@epi052
Copy link
Owner

epi052 commented Feb 26, 2024

no worries, it sounds like there's not an easy static value with which we can warn folks. in that case, going to skip the warning and let current documentation/issues stand as-is.

Thank you for your help with this!

@epi052 epi052 closed this as completed Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants