-
Notifications
You must be signed in to change notification settings - Fork 13.8k
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
Enable word wrapped rex tables by default #14670
Enable word wrapped rex tables by default #14670
Conversation
773b3df
to
93cbb92
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All looks good, minus the one part on the open3
import which should really be a seperate PR, and which now seems to be being tracked at #14674. If you want to remove this edit from this PR so we can track it appropriately in #14674, then that should be the only change needed.
Will proceed with testing this now.
lib/metasploit/framework/data_service/remote/managed_remote_data_service.rb
Outdated
Show resolved
Hide resolved
Converting this to a draft; now that I think about it - I'd prefer to skip today's release, and ensure it doesn't have any issues with Pro. |
Confirmed that the Creds Output Wraps
Also
However other parts like Tables In Info Output Wrap, But Not Rest of The Text
That being said as far as the tables go within these outputs they are wrapping as intended, as can be seen with Options Wrapping Output
So looks like this PR is working in so far as what it claims to do 👍, but we should be aware there is likely more work to be done here to get other sections of text to wrap properly and to give a consistent user experience. |
Oh woops one more confirmation but did confirm that setting this option to false does indeed revert the wrapping behavior as intended: Setting wrapped_tables to false works successfully
|
Tested this with Pro on a local development environment on osx, and it shows that the wrapped tables functionality depends on the engine's terminal width server side - rather than the browser's console width. This can be shown in the below screenshot. First the Note: I am not replicating the exact production environment locally, I will have to retest this and confirm its behavior. Or make code changes to support this correctly. Update: Production environments work as expected, most likely due to the work of rapid7/rex-text#38 |
93cbb92
to
c000041
Compare
c000041
to
f3ce908
Compare
Testing after rebase shows the results are the same, no issues with functionality being affected. Will go ahead and land this now. |
Release NotesUpdated Rex tables to have word wrapping enabled by default for all Rex tables (except for those output by the |
Enables word wrapped rex tables by default, apart from the following potentially workflow breaking areas:
use <paste-buffer>
commandIn a future PR these other above areas will be enabled too, once it's confirmed that this PR doesn't break other use cases/tooling.
Before
After
Verification
features set wrapped_tables false
disables functionality