セキュリティ対応のために依存ライブラリのバージョンアップデート #1
Merged
Conversation
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
moonstruckdrops
requested review from
KoichiOzaki,
masarufuruya,
morooka-akira and
tomoyukikat0
tomoyukikat0
approved these changes
Jul 12, 2024
moonstruckdrops
added a commit
that referenced
this pull request
Jul 12, 2024
This reverts commit 070feb1.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
概要
弊社が使用しているGoogleDrive/SpreasSheetのgemは、公式のgemを使用しているわけではなく、サードパーティが作ったgemになっています
そのgemがメンテされておらず、特に認証・認可部分のライブラリアップデートのPRが滞っています
gimite#419
この結果、弊社が使用しているGoogle APIのgem、または関連するgemが軒並みアップデート不可能という状況に陥っています
本来であれば、メンテがされていないためにこのgemを外して公式のものに対応するのが一番良いですが、
まずはこのgemをforkして、件のPR内容を取り込めるようにします
こうすることで、他のGoogle APIのgemをアップデートできるようにします
ライセンスとしても修正BSDなので、この改修によって特に大きな影響は無い(バイナリにして再配布もしない)です
やったこと
# この内容を取り込み次第、linebot-service側の対応も行います
チケット
https://www.notion.so/pharma-x/google_drive-gem-5e7851dd85db45939d352b323f680b39