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

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

Open
kazuminn opened this issue May 7, 2023 · 6 comments
Open

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

kazuminn opened this issue May 7, 2023 · 6 comments
Assignees

Comments

@kazuminn
Copy link
Collaborator

kazuminn commented May 7, 2023

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

@kazuminn
Copy link
Collaborator Author

kazuminn commented May 7, 2023

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

@YamatoSecurity
Copy link
Collaborator

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

@YamatoSecurity
Copy link
Collaborator

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

@kazuminn
Copy link
Collaborator Author

@fukusuket @hitenkoku 相談ですが、

src/detection/rule/utiles.rs

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

@fukusuket
Copy link
Collaborator

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

@hitenkoku
Copy link
Collaborator

hitenkoku commented May 28, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants