-
Notifications
You must be signed in to change notification settings - Fork 29.3k
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
Sandbox background process architecture #116337
Comments
Won't this delay terminal creation for several seconds after start up? That was the main reason for keeping this out of Shared Process. Is this temporary based on whether we can make it node or when we get message port support would be move it to be a child of main? |
@Tyriar once we let the shared process be the parent of the vscode/src/vs/code/electron-main/app.ts Lines 458 to 462 in e371fae
|
After discussion in the Electron sync yesterday, here is a revised diagram given @deepak1556 feedback that the new process architecture will not provide |
Merged into #92164. |
From discussion today in standup here are suggestions for the sandbox architecture:
MessagePort
support and access to Electron's proxy solution for network requests (if we cannot get proxy support here, we could think about doing ad-hoc hidden windows for network requests as a workaround)The text was updated successfully, but these errors were encountered: