-
Notifications
You must be signed in to change notification settings - Fork 200
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
Deleted return characters in output of search command #1018
Deleted return characters in output of search command #1018
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #1018 +/- ##
==========================================
- Coverage 73.89% 73.87% -0.03%
==========================================
Files 25 24 -1
Lines 17792 17784 -8
==========================================
- Hits 13148 13138 -10
- Misses 4644 4646 +2
☔ View full report in Codecov by Sentry. |
@hitenkoku ありがとうございます!
|
@YamatoSecurity 確認ありがとうございます。searchの方は連続した空白は一つの空白となるように変更してみていますはずですが、どこかで問題ありましたでしょうか? |
csv-timelineでは行っていなかったようなのでissue新しく切ってそちらで管理させていただければと思います |
@hitenkoku あ、なるほど。 元のAccessListフィールドを調べたら、スペースが一個になっているように見えますが、Hayabusaの内部では間にタブ文字が入ってたりしますか?(確か、間にタブや改行文字が入っていた気がしました) |
ご指摘ありがとうございます。今回対処した範囲だとCSVの出力のみ対応していました……画面出力についてもすぐに対応します |
ご指摘いただき誠にありがとうございます。頂いたコマンドを元に d1a957c で修正して画面表示でも複数の空白を1つの空白にまとめる形で対応完了しました! |
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.
@hitenkoku そういうことだったんですね。標準出力を確認したら、LGTM!
良いと思います。ありがとうございました!
ChangelogとHayabusaバージョンを更新しておきました。
ありがとうございます!修正いただきありがとうございます。修正いただいた内容で問題ないと思います。それではマージさせていただきます! |
What Changed
Evidence
Please describe the functionality gained by merging this pull-request and the evidence that the bug has been fixed.
main branch output
PS > ./main.exe search -d ..\hayabusa-sample-evtx\ --keywords "null" -o main.csv
...
Total event log files: 584
Total file size: 137.1 MB
584 / 584 [==========================================================================] 100.00 %
Total findings: 44144
Elapsed time: 00:00:09.375
File Contents
I would appreciate it if you could review.