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

Fix search command highlighting for single characters #14930

Conversation

adfoster-r7
Copy link
Contributor

@adfoster-r7 adfoster-r7 commented Mar 22, 2021

closes #14922

Thanks to @pingport80 for raising the issue and pointing to where the code was having issues over a quick slack chat

Before

image

After

image

Verification

List the steps needed to make sure this thing works

  • Open msfconsole
  • search a eternal
  • Verify that the results are highlighted as expected

@adfoster-r7 adfoster-r7 force-pushed the fix-search-command-for-single-characters branch from 33ea973 to b8d9e6d Compare March 22, 2021 19:01
@adfoster-r7 adfoster-r7 force-pushed the fix-search-command-for-single-characters branch from b8d9e6d to d63ff8a Compare March 22, 2021 19:02
@adfoster-r7 adfoster-r7 changed the title Fix search command for single characters Fix search command highlighting for single characters Mar 22, 2021
@smcintyre-r7 smcintyre-r7 self-assigned this Mar 22, 2021
@smcintyre-r7
Copy link
Contributor

Verified that I could reproduce the original issue and that this fixes it. Quick note, that it didn't look like all single letter queries would reproduce the error, but that at least a would.

image

I'll have this landed shortly, thanks @adfoster-r7 !

@smcintyre-r7 smcintyre-r7 merged commit 20ae05f into rapid7:master Mar 22, 2021
@smcintyre-r7
Copy link
Contributor

smcintyre-r7 commented Mar 22, 2021

Release Notes

Fixed a bug where the highlighting in msfconsole's search command would break when the search term was certain single letter queries.

@adfoster-r7 adfoster-r7 deleted the fix-search-command-for-single-characters branch March 23, 2021 13:12
@cgranleese-r7 cgranleese-r7 added the rn-fix release notes fix label Mar 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug library rn-fix release notes fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Search command not working as intended for single letters
3 participants