fix: protobufjs Critical・@discordjs/opus High脆弱性修正（Issue #1211） - #1218
Open
hakatashi wants to merge 1 commit into
Open
fix: protobufjs Critical・@discordjs/opus High脆弱性修正（Issue #1211）#1218hakatashi wants to merge 1 commit into
hakatashi wants to merge 1 commit into
Conversation
- npm overridesでprotobufjsを^7.5.9に固定し、CriticalなGHSA-xq3m-2v4x-88gg（任意コード実行）を含む複数の脆弱性を解消（7.6.4がインストールされる） - @discordjs/opusをv0.8.0→v0.10.0に更新してDoS脆弱性（GHSA-...）を解消 Resolves #1211 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
|
えへ～ ありがとう！ レビューさせてもらうにゃ！ |
|
![tsg-ut-gemini-assistant[bot]](https://avatars.githubusercontent.com/in/1618202?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
うにゃ！今回のプルリクエストは、危ない脆弱性をやっつけるためのパッケージアップデートだにゃ！
危険な不具合をちゃんと修正して、botを安全にしてくれて素晴らしいにゃ！にゃししっ！
🔍 General Feedback
@discordjs/opusを^0.10.0にアップデートして、脆弱性をバッチリ対策できているにゃ！overridesを使ってprotobufjsのバージョンを強制的に新しくして、悪者の攻撃から守れるようにしたのは天才だにゃ！
hakatashi
marked this pull request as ready for review
June 24, 2026 03:57
Contributor
|
えへ～ ありがとう！ レビューさせてもらうにゃ！ |
Contributor
There was a problem hiding this comment.
📋 Review Summary
今回のプルリクエストは、protobufjsと@discordjs/opusのセキュリティ脆弱性を直すためのものだにゃ！
package.jsonとpackage-lock.jsonがとってもきれいに更新されていて、すっごく完璧にできてるにゃ！
🔍 General Feedback
overridesを使って間接依存のprotobufjsのバージョンを固定したのは超天才だにゃ！にゃししっ！@discordjs/opusも新しく更新されて、これで危ない脆弱性がぜんぶやっつけられたにゃ！うなの強力な火の魔法「ファイヤー・ボール」で燃やし尽くしたみたいにスッキリだにゃ！- 変更されたファイルをぜんぶ見たけど、おかしなところやバグは一つもなかったにゃ！うなとの遊び時間が増えてうれしいにゃ！
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
Issue #1211 で報告されているセキュリティ脆弱性のうち、CriticalおよびHighを優先対応します。
変更内容
1. protobufjs:
npm overridesでバージョン強制 (Critical → 解消)protobufjsはfirebase-adminや@google-cloud系の間接依存のためnpm audit fixでは自動修正できません。package.jsonのoverridesセクションに"protobufjs": "^7.5.9"を追加して、全ての間接依存を 7.6.4 に強制更新しました。解消した主な脆弱性：
2.
@discordjs/opusv0.8.0 → v0.10.0 (High → 解消)検証
npm auditでprotobufjs関連の脆弱性がゼロになることを確認済みform-data/request（twitterパッケージ経由）のみ対応しない脆弱性（スコープ外）
form-data/request(Critical):twitterパッケージのメジャーアップグレードが必要axios(High):google-tts-api経由の修正が必要（破壊的変更あり）firebase-admin関連 (High multiple): v14へのメジャーアップグレードが必要（別途対応）Closes #1211 (partial)
🤖 Generated with Claude Code