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

Fix CSP error in Chrome 130+ #32

Merged
merged 1 commit into from
Dec 14, 2024
Merged

Conversation

sunya9
Copy link
Contributor

@sunya9 sunya9 commented Dec 14, 2024

便利な拡張機能ありがとうございます。

ここ最近、拡張機能がうまく動作していないことに気付きました。
調査をしてみるとChromeの更新に伴うCSP周りの挙動の変更によるものだと判明しました。

crxjs周辺のツールを更新することで対処できるそうなので、そのように対処しました。
ツールチェインの更新に伴い幾分表記が変わっている部分がありますが、本質的な部分である内容自体の変更はありません。

よろしくお願いします。

@r7kamura r7kamura added the fix Fix bug. label Dec 14, 2024
@r7kamura
Copy link
Owner

ありがとうございます!
ひとまず手元でビルドして試してみます

@r7kamura
Copy link
Owner

おー、手元でも正しく動くようになることを確認できました。早速取り込んで、新しいバージョンをリリースしてみます。

@r7kamura r7kamura merged commit 85a50b2 into r7kamura:main Dec 14, 2024
1 check passed
@r7kamura r7kamura changed the title fix: CSP error in Chrome 130+ Fix CSP error in Chrome 130+ Dec 14, 2024
@r7kamura
Copy link
Owner

新しいバージョンを審査に提出しておきました。直るのが楽しみです

image

@sunya9
Copy link
Contributor Author

sunya9 commented Dec 16, 2024

無事リリースされたようです。迅速な対応ありがとうございました 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fix bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants