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.

Sign up for GitHub

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

Jump to bottom

実行コンテキストを安全に #20

Closed
ci7lus opened this issue Oct 9, 2021 · 3 comments · Fixed by #34
Closed

実行コンテキストを安全に #20

ci7lus opened this issue Oct 9, 2021 · 3 comments · Fixed by #34
Labels
enhancement New feature or request v2 v2に関連するIssue
Milestone
v2

Comments

@ci7lus
Copy link
Owner

ci7lus commented Oct 9, 2021 •
edited
Loading

現状実行しているElectronは、簡単のために安全のための措置をあらかた無効化してしまっている。
これでは問題なので、v2.0.0リリースまでにはある程度解消する。

参考: https://www.electronjs.org/docs/tutorial/security

現状有効にしている機能

      contextIsolation: false,
      nodeIntegration: true,
      enableRemoteModule: true,
@ci7lus ci7lus added enhancement New feature or request help wanted Extra attention is needed labels Oct 9, 2021
@ci7lus ci7lus changed the title カスタムヘッダー 実行コンテキストを安全に Oct 9, 2021
@ci7lus ci7lus added this to the v2 milestone Oct 9, 2021
@ci7lus ci7lus added the v2 v2に関連するIssue label Nov 17, 2021
@ci7lus
Copy link
Owner Author

ci7lus commented Dec 13, 2021 •
edited
Loading

remoteモジュール類と共存できると思ってたらターゲットをwebにする必要がある（electron12-preloadがwebpack-dev-serverに効かない？）のと、electron-store、ipcRenderer、remote全部をpreloadに押し込む必要がありかなりめんどくさい、webchimeraがこの方式で正しく動くかもわからないのにこの量の書き換えはうかつだった、というか別プロジェクトにwcjsだけ切り出してやればよかった

@ci7lus
Copy link
Owner Author

ci7lus commented Dec 14, 2021 •
edited
Loading

image
動作は確認 https://github.com/neneka/wcjs-ugly-demo

ci7lus added a commit that referenced this issue Dec 17, 2021
@ci7lus
🔒️ 実行コンテキストを安全に (#20)
d1b1825 
contextIsolation有効化/nodeIntegration無効化/remote依存削除
@ci7lus ci7lus removed the help wanted Extra attention is needed label Dec 31, 2021
@ci7lus
Copy link
Owner Author

ci7lus commented Dec 31, 2021

https://github.com/ci7lus/MirakTest/actions/runs/1640753617 これの動作チェックが取れ次第beta.3でリリースしたい

@ci7lus ci7lus mentioned this issue Dec 31, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request v2 v2に関連するIssue
Projects
None yet
Milestone
v2
Development

Successfully merging a pull request may close this issue.

実行コンテキストを安全に ci7lus/MirakTest
1 participant
@ci7lus