hayabusa icon indicating copy to clipboard operation
hayabusa copied to clipboard

Separate the contents of src/detections/rule/mod.rs into modules

Open kazuminn opened this issue 1 year ago • 6 comments

src/detections/rule/mod.rsの中に、DetectionNodeやRuleNodeが混じって入っていて、カオスになってきている気がする。 mod構文とそれらが入り混じっていて、読みにくく、管理しにくい問題がある。

kazuminn avatar May 07 '23 15:05 kazuminn

分けようと考えたけど、うまくモジュール構成を決められない。

kazuminn avatar May 07 '23 15:05 kazuminn

@kazuminn ありがとうございます!コードの整理も大事ですね。 一旦モジュール構成を考えてみて、 @fukusuket と @hitenkoku に見て貰うのはどうですか? 何か悩んでいることがあれば、ここで共有して下さい。

YamatoSecurity avatar May 07 '23 21:05 YamatoSecurity

@kazuminn ミーティングで話したところ、良い感じに分離できるのであれば、是非分離した方が良さそうなので、お願いしたいですが、他のissue(新機能等)と比べると優先度が低いという結論でした。と言いながら、気になって先にやっておきたいのであれば、全然良いです。 基本的にお任せします!

YamatoSecurity avatar May 14 '23 22:05 YamatoSecurity

@fukusuket @hitenkoku 相談ですが、

src/detection/rule/utiles.rs

を作っちゃって、mod xxx以外をそこに突っ込むのは、どうでしょう? DetectionNodeとRuleNodeとaggresultとテストなどをそこに移動する感じです。

kazuminn avatar May 27 '23 16:05 kazuminn

@kazuminn 良いと思います! あとAggresultをcount.rsあたりに移動できるのであれば、ファイル名もutiles.rsからrulenode.rsとかにできそうと思いました。(見当違いでしたらすみません!)

fukusuket avatar May 27 '23 20:05 fukusuket

@fukusuket 私もrulenode.rsにする案で問題ないと思います。。

hitenkoku avatar May 28 '23 06:05 hitenkoku