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

feat: 脆弱性診断結果をdiscordに通知させる #1209

Merged
merged 6 commits into from
May 7, 2024

Conversation

weweweok
Copy link
Contributor

@weweweok weweweok commented May 2, 2024

内容

依存パッケージ脆弱性診断の結果をdiscordに通知するために必要な、雛形のようなものを作ってみました。
この雛形を編集し、webhookのurlをsecretsの変数として定義することで、voicevoxコミュニティに脆弱性診断を送信できると思います。

関連 Issue

スクリーンショット・動画など

}]
} ' "ここにDiscordのWebhookURLを書く"

uses: sarisia/actions-status-discord@v1
Copy link
Contributor Author

@weweweok weweweok May 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

こちら、色々適切なものに書き換えていただけると幸いです。(webhooksのsecretsやurlなど)

設定の詳細はこちらに載っています

https://github.com/marketplace/actions/actions-status-discord

@weweweok weweweok marked this pull request as ready for review May 2, 2024 11:16
@weweweok weweweok requested a review from a team as a code owner May 2, 2024 11:16
@weweweok weweweok requested review from Hiroshiba and removed request for a team May 2, 2024 11:16
@weweweok weweweok changed the title feat: 脆弱性診断結果をdiscordに通知させるymlの雛形を作成 feat: 脆弱性診断結果をdiscordに通知させる May 3, 2024
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

プルリクエストありがとうございます!!
secretsは任せてください!

こちら以下の変更をお願いできますか・・・?

  • 診断結果(OKかダメか)をメッセージ内に含める
    • 直前のステップがfailしたかどうかでわかりそう
  • urlは走ったWorkflowのURLが良さそう

ChatGPTくんがいうにやり方はこんな感じとのことでした!↓
https://chat.openai.com/share/8fc22017-8d4c-44db-a22b-b23b47fbb5a7

@weweweok
Copy link
Contributor Author

weweweok commented May 4, 2024

自分のサーバーで動かしたときの結果です。失敗時と成功時で縦線の色を分けてみました。(画像の緑色と赤色部分)

image

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

サンプルもありがとうございます!!!
早速secretsを設定してみます!動くのは来週なはずなので楽しみですね・・・!

@Hiroshiba Hiroshiba merged commit 3df49b9 into VOICEVOX:master May 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants