-
Notifications
You must be signed in to change notification settings - Fork 210
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
Pivot Keyword List機能の追加 #412
Conversation
おっと、rebaseが必要になるので、少しお待ちください。 |
dba8765
to
20042f2
Compare
CI通りました。 @hitenkoku さん @YamatoSecurity さん。 |
2GBのBig-Security.evtxでスキャンかけましたが、あまり出てこない結果になりました。こんなもんなんですね。
|
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.
一応ソースコードレベルで気になるところを指摘しました。まだ動作確認はしていません。
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.
記載が漏れておりましたが、documentも日本語だけでもよいので作成してください。
英語版は @YamatoSecurity に添削してもらうなりしましょう。
レビューの指摘の対応は終えたので、 |
4ca4955
to
c1e0d0e
Compare
一応、自分の範囲のclippyのwarnは直した。(clippyのプルリクが先にマージされると、また、再度rebaseが必要になってきそう。) |
@hitenkoku @YamatoSecurity レビューの修正とclippyのwarn処理は終わりました。 |
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.
修正分も含めて、確認しました。いくつか指摘をしましたのでご確認ください。
ドキュメント作成と機能の作成を分けてしまうと、ドキュメントの更新が遅れて、develop側の実態が合わなくなってしまう可能性が出てくるので、ドキュメントを追加した上でマージ対応をさせてもらったほうが後々良いと思いますので、このプルリクエストはドキュメントを書いてからマージをするようにしましょう。
@hitenkoku @YamatoSecurity え、keyは固定でfields(%hoge%のやつ)が自由に設定できる仕様の話だったと思います。
(keyをconfigでカスタマイズする実装することもできる気はしますが) |
@YamatoSecurity |
issueでは固定である旨を書いていません。あと @YamatoSecurity はissueで以下のように書いています。単純に読めば、項目のつ追加対応が容易にできるようにするほうが良いと思います。 @YamatoSecurity 本件、念の為ご確認をお願いいたします。
|
@kazuminn が以下のように発言をまとめています。 @YamatoSecurity はキーワードは任意で読み込めるようにという依頼をしているようです。
|
返事が遅くなってすみません! はい、キーも自由に指定したいです。
はキーも含めて全部カスタマイズできると思っていました。 また、色々追加の注文がありまして、すみません。。 今の画面出力:
を
という風に出力できますか?
また、 例えば |
@YamatoSecurity たしかに、その方がtimelineを絞れて、良さそうです。 |
すまない! m(__)m |
@kazuminn |
ドキュメントの作成者は誰でも良いのですが、ドキュメントはpull requestをマージする状態にはすでにかけてるといいなぁと思っています。(というか書けていないと、結局ドキュメント作成が後手に回ってしまうと大変なので……) |
@YamatoSecurity |
Dさんが言っていたように、configファイルを以下のように変更します。
|
@YamatoSecurity
|
@YamatoSecurity デフォルトの設定ファイルはこれでよいですか?
|
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.
OKです!ありがとうございます!
ドキュメンテーションお待たせしました。書いてみました。 @hitenkoku お手数をかけますが、確認&和訳をお願いできますか? |
おっと、こちらの環境では出力メッセージを編集しても問題なかったけど、githubではbuild failedになっています。誰か確認して頂けますか? |
@YamatoSecurity 英語ドキュメントありがとうございます。 |
@YamatoSecurity 時間短縮ために、 今の使い方の、 前に、そんな感じのことを会議でyamatosecurityさんが言ったときは、Dさんが複雑になると言っていましたが、今やろうと思うと、そんなに複雑ではないのでは、と思いました。 -oはcsvを吐いてくれて、-pを付けると、キーワードファイルも吐ける。みたいに、ユーザーも理解しやすいと思います。 リリースも近いので、次のバージョンに回しても大丈夫です。 |
前回話した際は-pをつけた場合と-pをつけなかった場合で出力されるものが異なる(その時は-pをつけていたらここで話をしているpivot keywordlistの出力。-pをつけていなかったら今までの結果のcsvの出力)となっている場合にusageのときに説明がつけづらいと考えていました。 -p をつけていてもつけていなくても今までのcsvが出力されるのであれば -p側のusageに 「-oを追加すれば別途pivot keyword listのcsvを出力する」旨を書けば特に混乱はないと思っています。 |
私は特に決定権ないので @YamatoSecurity に一任します |
@kazuminn 提案ありがとうございます! @hitenkoku ご意見ありがとうございます!確かに勘違いする人も居るかもしれないので、最後のメッセージの出力時に「Saved a timeline filtered by the pivot keywords found.」等を追加しておけば、伝わると思います。 |
@YamatoSecurity 遅くなるので、たしかに、オプションのほうが良いと思いました。 もしやるなら、別のプルリクでやりましょうか。 |
そうですね。 |
@YamatoSecurity ドキュメントを確認しました。特に問題はないと思います。また、 6009779 で日本語訳を追加しましたのでご確認のほど、よろしくお願いいたします。 |
日本語訳ありがとうございます。LGTM |
@hitenkoku ありがとうございます!一箇所だけ分かりにくいところがあったので、修正してみました。ご確認ください。 |
@YamatoSecurity 確認いたしました。修正ありがとうございます。問題ないと思いますのでマージ対応をしておきます |
* Fix/fix clippy warn (#434) - Fixed following Clippy Warnings(previous warning count: 671 -> after: 4) - clippy::needless_return - clippy::println_empty_string - clippy::redundant_field_names - clippy::single_char_pattern - clippy::len_zero - clippy::iter_nth_zero - clippy::bool_comparison - clippy::question_mark - clippy::needless_collect - clippy::unnecessary_unwrap - clippy::ptr_arg - clippy::needless_collect - clippy::needless_borrow - clippy::new_without_default - clippy::assign_op_pattern - clippy::bool_assert_comparison - clippy::into_iter_on_ref - clippy::deref_addrof - clippy::while_let_on_iterator - clippy::match_like_matches_macro - clippy::or_fun_call - clippy::useless_conversion - clippy::let_and_return - clippy::redundant_clone - clippy::redundant_closure - clippy::cmp_owned - clippy::upper_case_acronyms - clippy::map_identity - clippy::unused_io_amount - clippy::assertions_on_constants - clippy::op_ref - clippy::useless_vec - clippy::vec_init_then_push - clippy::useless_format - clippy::bind_instead_of_map - clippy::bool_comparison - clippy::clone_on_copy - clippy::too_many_arguments - clippy::module_inception - fixed clippy::needless_lifetimes - fixed clippy::borrowed_box (Thanks for helping by hach1yon!) * Merge main and output fix#443#444 (#445) * removed tools/sigmac (#441) * removed tools/sigmac - moved tools/sigmac to hayabusa-rules repo * fixed doc link tools/sigmac * fixed submodule track * fixed submodule track from latest to v1.1.0 tag * fixed link * erased enter #444 * erased enter #444 * reverted logo enter * fixed rules submodule target commit #444 Co-authored-by: Yamato Security <[email protected]> * readme update screenshots etc (#448) * Opensslを静的にコンパイルするためにCargo.tomlの設定変更 (#437) * cargo update - openssl static * updated cargo * macos2apple * cargo update * cargo update * aliasキーがない場合もEvent.EventDataを自動で走査する (#442) * add no event key * support not-register-alias search * added checking EventData when key do not match in alias #290 - added checking key in Event.EventData, if key is not exist in eventkey_alias.txt. * cargo fmt * fixed panic when filter files does not exists * fixed errorlog format when filter config files does not exist Co-authored-by: DustInDark <[email protected]> * changed downcast library from mopa to downcast_rs #447 (#450) * Fixed Clippy Warnings (#451) * fixed clippy warn * fixed cargo clippy warnging * fixed clippy warngings in clippy ver 0.1.59 * fixed clippy warnings clippy::unnecessary_to_owned * added temporary blackhat arsenal badge * added rust report card badges #453 * added repository maintenance levels badge #453 * documentation update macOS usage etc * update * added clippy workflow #428 (#429) * added clippy workflow #428 * fixed action yaml to run clippy #428 * fixed indent * fixed workflow * fixed workflow error * fixed indent * changed no annotation #428 * adujusted annotation version * fixed clippy::needless_match * remove if let exception * removed unnecessary permission check #428 * statistics event id update (#457) * Feature/#440 refactoring #395 (#464) * updated submodule * fix degrade for pull req #464 (#468) * fix degrade for pull req #464 * add trim * Fearture/ added output update result#410 (#452) * add git2 crate #391 * added Update option #391 * updated readme #391 * fixed cargo.lock * fixed option if-statement #391 * changed utc short option and rule-update short option #391 * updated readme * updated readme * fixed -u long option & version number update #391 * added fast-forwarding rules repository #391 * updated command line option #391 * moved output logo prev update rule * fixed readme #391 * removed recursive option in readme * changed rules update from clone and pull to submodule update #391 * fixed document * changed unnecessary clone recursively to clone only * English message update. * cargo fmt * English message update. ( 4657c35 cherry-pick) * added create rules folder when rules folder is not exist * fixed gitmodules github-rules url from ssh to https * added output of updated file #420 * fixed error #410 * changed update rule list seq * added test * fixed output #410 * fixed output and fixed output date field when modified field is lacked #410 * fixed compile error * fixed output - added enter after Latest rule update output - added output when no exist new rule - fixed Latest rule update date format - changed output from 'Latest rule update' to 'Latest rules update' * fixed compile error * changed modified date source from rules folder to each yml rule file * formatting use chrono in main.rs * merge develop clippy ci * fixed output when no update rule #410 - removed Latest rule update - no output "Rules update successfully" when No rule changed * Change English Co-authored-by: Tanaka Zakku <[email protected]> * Remove unnecessary code from timeline_event_info and rename files for… (#470) * Remove unnecessary code from timeline_event_info and rename files for issue462 * Remove unnecessary code #462 * add equalsfield pipe (#467) * Enhancement: add config config #456 (#471) * added config option #456 * added process of option to speicifed config folder #456 following files adjust config option. * noisy_rules.txt * exclude_rules.txt * fixed usage in readme * updated rules submodule: * fixed process when yml file exist in .git folder * ignore when yml file exist in .git folder * Add: --level-tuning option's outline * Add: read Rule files * Add: input rule_level.txt files & read rules * cargo fmt * Add: level-tuning function * Reface: split to options file * WIP: Text overwrite failed... * Fix: Text overwrite was failed * Add: Error handlings * Add: id, level validation * mv: IDS_REGEX to configs file * fix: level tuning's file name * Cargo fmt * Pivot Keyword List機能の追加 (#412) * add get_pivot_keyword() func * change function name and call it's function * [WIP] support config file * compilete output * cargo fmt * [WIP] add test * add test * support -o option in pivot * add pivot mod * fix miss * pass test in pivot.rs * add comment * pass all test * add fast return * fix output * add test config file * review * rebase * cargo fmt * test pass * fix clippy in my commit * cargo fmt * little refactor * change file input logic and config format * [WIP] change output * [wip] change deta structure * change output & change data structure * pass test * add config * cargo fmt & clippy & rebase * fix cllipy * delete /rules/ in .gitignore * clean comment * clean * clean * fix rebase miss * fix rebase miss * fix clippy * file name output on -o to stdout * add pivot_keywords.txt to ./config * updated english * Documentation update * cargo fmt and clean * updated translate japanese * readme update * readme update Co-authored-by: DustInDark <[email protected]> Co-authored-by: Tanaka Zakku <[email protected]> * Add: test * Add: README.md * Cargo fmt * Use #[cfg(test)] * Fixed output stop when control char exist in windows terminal (#485) * added control character filter in details #382 * fixed document - removed fixed windows teminal caution in readme * fixed level tuning test and added test files #390 * changed level_tuning.txt header from next_level to new_level * fixed convert miss change to low level * added run args rules path to check test easy #390 * fixed comment out processing in level_tuning.txt * fixed config to show level-tuning option * fixed level-tuning option usage from required to option * reduce output mitre attack detail tachnique No. by config file (#483) * reduced mitre attck tag output by config file #477 * prepared 1.2.0 version toml * added test files and mitre attck strategy tag file #477 * fixed cargo.toml version * updated cargo.lock * output tag english update * cargo fmt Co-authored-by: Tanaka Zakku <[email protected]> * Fix: test file's path was incorrect * Add: add test_files/config/level_tuning.txt * Add: Flush method. * inserted debug data * reverted config usage * fixed test yaml file path * Feature/#216 output allfields csvnewcolumn (#469) * refactoring * refactoring * under constructing * underconstructing * under construction * underconstructing * fix existing testcase * finish implement * fmt * add option * change name * fix control code bug * fix disp * change format and fix testcase * fix help * Fix: show usage when hayabusa has no args * rm: debug line * Enhance/warning architecture#478 (#482) * added enhance of architecture check #478 * changed check architecture process after output logo #478 * English msg update * fixed detect method of os-bit to windows and linux * removed mac and unix architecture and binary and updated its process of windows * fix clippy * added check on Wow64 env #478 * Update contributors.txt Co-authored-by: Tanaka Zakku <[email protected]> * added --level-tuning option to usage * Revert "added --level-tuning option to usage" This reverts commit e6a7409. * readme update * Update README-Japanese.md * readme, version, cargo update * typo fix * typo fix * rm: duplicated test & fix test name * Add: show logo, and some infos * small english fix * twitter link fix (#486) * added feature of tag output reducing to agg condition #477 (#488) * changed level output from informational to info #491 * updated rules submodule * v1.2 changelog update (#473) * changelog update * Update CHANGELOG.md added contributor in "Fields that are not defined in eventkey_alias.txt will automatically be searched in Event.EventData." ref #442 * Update CHANGELOG-Japanese.md Fields that are not defined in eventkey_alias.txt will automatically be searched in Event.EventData. added contributor in "Fields that are not defined in eventkey_alias.txt will automatically be searched in Event.EventData." ref #442 * Update CHANGELOG.md added bug fixes (#444) and `Performance and. accuracy` add contributor ref(#395) * Update CHANGELOG-Japanese.md * Translated v1.2 change log to Japanese v1.2の内容を日本語に修正 * fixed typo added lacked back quote. * added description added following issue and pr description to readme - #216 / #469 L8 - #390 / #459 L9 - #478 / #482 L19 - #477/ #483 L20 * added description README.md added following issue and pr description to readme - #216 / #469 L8 - #390 / #459 L9 - #478 / #482 L19 - #477/ #483 L20 * changelog update * changelog update * update Co-authored-by: DustInDark <[email protected]> * updated rules #493 (#494) * Resolve conflict develop (#496) * removed tools/sigmac (#441) * removed tools/sigmac - moved tools/sigmac to hayabusa-rules repo * fixed doc link tools/sigmac * fixed submodule track * fixed submodule track from latest to v1.1.0 tag * fixed link * fixed rules submodule targe #444 * updated submodule * updated rules submodule Co-authored-by: Yamato Security <[email protected]> Co-authored-by: Yamato Security <[email protected]> Co-authored-by: kazuminn <[email protected]> Co-authored-by: James / hach1yon <[email protected]> Co-authored-by: garigariganzy <[email protected]> Co-authored-by: itiB <[email protected]>
fix #393.
なにこれ
(levelがlow以上の)アラートがある場合は、以下のフィールドから"Pivot Keyword"(攻撃者の足跡になる送信元のIPアドレス、ログオンID等々)をリストアップして、出力する機能です。キーワードリストで、大量のinformationalログをgrepして、攻撃者の痕跡だけを抽出するための機能です。(informationalログが多すぎて解析が難しいという意見(クレーム?)が来ていたので、これで不要なinformationalログが減るはずです。)
また、キーワードの一覧を見るだけで、どのユーザや端末が攻撃を受けている可能性があるかパッと見て分かるので、fast forenscisに役立ちます。
以下のように実装しました。
オプション: -p / --pivot-keywords-list -o <ファイル名> or -p
(ただの、-oだけだと、csv出力になる。)
例: ./hayabusa.exe --min-level high --pivot-keywords-list -o keywords.txt -d ./sample-evtx
-oがあると、スキャン後にファイルに出力する。
-oがないと、タイムラインではなく、標準出力する。
hashSetをつかっているので、キーワードが重複することはない。
空("-")、("127.0.0.1")、("::1")になっている可能性があって、誤検知になりそうなので、すべてで無視している。
キーワードをハードコーディングではなく、設定ファイルで自由に修正できるようにしました。config/pivot_keywords.txtで設定できます。
configをcsvとして読み込む。すべての行数が、一致しなくてもよい。flexibleになっている。
取り敢えず、アラートに以下のフィールドが存在する場合は、Pivot Keywords Listに追加します。(デフォルトでのconfig設定)
evidence
手動テスト項目
(ちなみに、コードにおいて、-oオプションの依存度が複雑になってくるので、今後手を入れる場合は、何か対策が必要になってくる。)