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.

Sign up for GitHub

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

Jump to bottom

Change Event ID Statistics wording to Metrics #706

Closed
YamatoSecurity opened this issue Sep 25, 2022 · 5 comments · Fixed by #708
Closed

Change Event ID Statistics wording to Metrics #706

YamatoSecurity opened this issue Sep 25, 2022 · 5 comments · Fixed by #708
Assignees
@hitenkoku
Labels
enhancement New feature or request
Milestone
v1.7.0

Comments

@YamatoSecurity
Copy link
Collaborator

-s, --statisticsオプション名ですが、英語でこういう集計のことをstatisticsという人が多いが、よく考えたら本当はmetricsの方が正しいと思うので、-M, --metricsに変えたいです。混乱しないようにソースコードの方でもstatisticsをmetricsに変えた方が良さそうです。
@YamatoSecurity YamatoSecurity added the enhancement New feature or request label Sep 25, 2022
@hitenkoku hitenkoku self-assigned this Sep 25, 2022
@hitenkoku
Copy link
Collaborator

対応します。こちらはオプションの文面は変更なしで良いでしょうか。

オプション名変更に伴いファイル名の方もmetricsに変えておこうと思います

@YamatoSecurity
Copy link
Collaborator Author

ありがとうございます！はい、基本的にstatisticsという単語を全部metricsに変えたいです。

@hitenkoku
Copy link
Collaborator

了解しました。それであればソースコードのファイル名変更も含めて対応しておきます

@hitenkoku hitenkoku added this to the v1.7.0 milestone Sep 25, 2022
@hitenkoku
Copy link
Collaborator

一部そのままにしておいたほうが良さそうなファイルがありましたので共有しておきます。

rules内のstatistics_event_info.txtはrulesレポジトリで管理をしているので、metrics_event_info.txtに名前を変えてしまうと、
ユーザがrulesをアップデートしていないとエラーが出るようになってしまうのでrulesで管理しているconfigファイルはrenameの対象外とさせていただければと思います。

hitenkoku added a commit that referenced this issue Sep 25, 2022
@hitenkoku
changed Event ID Statistics wording to Metrics #706
08c6dcf
hitenkoku added a commit that referenced this issue Sep 25, 2022
@hitenkoku
renamed statistics.rs to metrics.rs #706
345133d
hitenkoku added a commit that referenced this issue Sep 25, 2022
@hitenkoku
changed metric short option from m to M #706
1c1297b
hitenkoku added a commit that referenced this issue Sep 25, 2022
@hitenkoku
updated changelog #706
578b497
hitenkoku added a commit that referenced this issue Sep 25, 2022
@hitenkoku
updated readme to adjust changed -s option #706
6d23620
@hitenkoku hitenkoku mentioned this issue Sep 25, 2022
Merged
@hitenkoku hitenkoku linked a pull request Sep 25, 2022 that will close this issue
Changed event id statistics wording to metrics #708
Merged
hitenkoku added a commit that referenced this issue Sep 25, 2022
@hitenkoku
fixed english readme #706
9ed5785
@YamatoSecurity
Copy link
Collaborator Author

なるほど。statistics_event_info.txtをすぐ消すと良くないですね。取り敢えず新しく使うevent_id_info.txtを用意しました。
Yamato-Security/hayabusa-rules#121

互換性のために２つのファイルを置いて、３ヶ月後?にstatistics_event_info.txtを消しましょうか？

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hitenkoku hitenkoku

Labels
enhancement New feature or request
Projects
None yet
Milestone
v1.7.0
Development

Successfully merging a pull request may close this issue.

Changed event id statistics wording to metrics Yamato-Security/hayabusa
2 participants
@hitenkoku @YamatoSecurity