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

Missing Long Output #161

Closed
ash34 opened this issue Jan 2, 2021 · 13 comments
Closed

Missing Long Output #161

ash34 opened this issue Jan 2, 2021 · 13 comments
Assignees
Labels
🐛 bug Something isn't working
Projects
Milestone

Comments

@ash34
Copy link

ash34 commented Jan 2, 2021

Not all output is shown if the output is more than the screen on-show. The top part of the output is lost when you hide the keyboard. (The part that did not fit onto the half-screen)

  • Device : iPad 7 gen
  • OS : iOS 14.3
  • WebSSH Pro version : 14.9.3

AB123E96-D206-47FB-927B-C975FB7EA07B
14A00C0B-9BA0-4A00-BDEC-BB1DB5C3096C

Third screenshot shows that the issue is not seen if the second command is repeated.

E3372CC5-E31C-432A-BA8A-90C3FCD34DF4

Steps to reproduce the behavior:

  1. Already have a good amount of output from a previous command (apt upgrade was used in the test)
  2. Enter a command that is known to give a lot of output (mediainfo was used in the test)
  3. Hide the keyboard with the key bottom-right
  4. See the top of the second command output is missing
@isontheline isontheline self-assigned this Jan 2, 2021
@isontheline isontheline added this to the 14.9 - Helicopter milestone Jan 2, 2021
@isontheline
Copy link
Owner

isontheline commented Jan 2, 2021

Hello @ash34 👋

Could you check again 14.7 fallback? (It is a new bug or was already there?)

Have a nice day ☀️

@ash34
Copy link
Author

ash34 commented Jan 2, 2021

Cannot reproduce in 14.7.1 or 14.7.4 (OK the hide keyboard key does not work in those versions but scrolling down also showed the issue in the original report.

@isontheline
Copy link
Owner

Please :

  1. Edit your connection
  2. Edit terminal settings
  3. Disable "reverse wrap-around"

The trouble is from #85 enabled by default
I will disable it by default for 14.9.4

Thank you a lot for your issue @ash34

@isontheline isontheline added 🐛 bug Something isn't working and removed triage-needed labels Jan 2, 2021
@isontheline isontheline added this to Done in SSH Jan 2, 2021
@isontheline
Copy link
Owner

14.9.4 will also be a release candidate for the App Store

@ash34
Copy link
Author

ash34 commented Jan 2, 2021

Confirmed. Disable the reverse wrap-around solves it.

@isontheline
Copy link
Owner

Thank you a lot @ash34 👍

@ash34
Copy link
Author

ash34 commented Jan 4, 2021

Actually this still happens even when Reverse Wrap-Around is disabled. Screenshot attached from earlier
1D6A8E0C-C02F-4F14-82D9-C320DE354D7C
today.

@isontheline
Copy link
Owner

With your provided steps I'm not able to reproduce it with wrap-around disabled :

Capture d’écran 2021-01-04 à 21 35 52

The history is not "eaten" by the second command on my side :(

@ash34
Copy link
Author

ash34 commented Jan 5, 2021

Try a cat of a complete syslog for example, then another command which produces more than the half-screen of output. It’s repeatable for me this way. Screenshots attached. I think I can prevent it with lots of empty lines.

Not sure if it is a bug or not or related or not but sometimes when exiting nano I find a previous command already on the input line but the cursor is at the start of it. Will raise it separately when it happens again.

B15B8C6E-63DC-4E1F-A3DC-6523044AB6C2
AE7A3EF0-1AA5-4F3C-AC23-34C4CCC7F979
5A0F6355-D68C-4571-8C86-2ADB13813F5A
9405AF58-6434-45AB-90D0-EECB9D5F50FF
AA62282A-ED50-4F0A-A938-045F18507630
0B96754E-9712-494C-B977-B7B06E554E37

@isontheline
Copy link
Owner

Thank you a lot @ash34 for your clear reproduce steps!

I will try as soon as possible and come back to you shortly.

@isontheline isontheline reopened this Jan 5, 2021
@isontheline
Copy link
Owner

isontheline commented Jan 6, 2021

I'm sorry I can't reproduce :(

Even with a very long output of 500KB :

wget -O apple.txt "https://fr.wikipedia.org/wiki/Apple"
cat apple.txt
cat /proc/cpuinfo

image

@isontheline isontheline moved this from Done to In progress in SSH Jan 6, 2021
@isontheline
Copy link
Owner

Are you able to reproduce with these commands ?

wget -O apple.txt "https://fr.wikipedia.org/wiki/Apple"
cat apple.txt
cat /proc/cpuinfo

@ash34
Copy link
Author

ash34 commented Jan 18, 2021

Are you able to reproduce with these commands ?

wget -O apple.txt "https://fr.wikipedia.org/wiki/Apple"
cat apple.txt
cat /proc/cpuinfo

No, cannot reproduce.

@isontheline isontheline moved this from In progress to Done in SSH Jan 27, 2021
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
No open projects
SSH
Done
Development

No branches or pull requests

2 participants