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

定期脆弱性診断の結果をDiscordに通知 #1204

Closed
3 tasks
takejohn opened this issue May 1, 2024 · 2 comments
Closed
3 tasks

定期脆弱性診断の結果をDiscordに通知 #1204

takejohn opened this issue May 1, 2024 · 2 comments

Comments

@takejohn
Copy link

takejohn commented May 1, 2024

内容

#1152 で追加された定期脆弱性診断の実行時、Discord にテストの結果を送信しましょう。

#1152 (review)

Pros 良くなる点

セキュリティテストが回ったことに気づくことができるようになる。

Cons 悪くなる点

なし

実現方法

GithHub Actions から curl コマンドで Discord の Webhook を叩くか、
既存の action を使用する。
Webhook URL は secrets にもっておくようにする。
該当 workflow: https://github.com/VOICEVOX/voicevox_engine/blob/master/.github/workflows/test-security.yml

VOICEVOXのバージョン

0.?.0

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

@Hiroshiba
Copy link
Member

個人的にはwebhook経由でworkflowからDiscordに通知を飛ばすの、ありなのではと思ってます!
そもそもDiscordのgithubチャンネルにいろいろ飛ばしてますし。

ただ永久にメンテし続けるべきかというとそうではないとも思ってます。
とりあえず使い心地を調べてみるために、せっかくだから1回実装してみるのも面白いのかなと。
(こう、月1回の小さいお祭りみたいな感じになれば楽しいのかなとか思ってます)

@tarepan tarepan added the 状態:実装者募集 実装者を募集している状態 label May 1, 2024
@tarepan tarepan added 状態:実装 実装をおこなっている状態 and removed 状態:実装者募集 実装者を募集している状態 labels May 6, 2024
@tarepan
Copy link
Contributor

tarepan commented May 15, 2024

通知が正常に機能したことを確認しました (Actions, Discord)。
@weweweok さん実装ありがとうございました!

@tarepan tarepan closed this as completed May 15, 2024
@tarepan tarepan removed the 状態:実装 実装をおこなっている状態 label May 15, 2024
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

3 participants