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

Positrons pops up "code-oss-dev Safe Storage" dialog when launched from within VSCode #2274

Open
wesm opened this issue Feb 16, 2024 · 6 comments
Labels
area: builds Issues related to Builds category.
Milestone

Comments

@wesm
Copy link
Contributor

wesm commented Feb 16, 2024

This comes up frequently when launching Positron from within VSCode

image
@juliasilge
Copy link
Contributor

This is the same as #1688

I think we have confirmed this only occurs for dev builds (does not impact users) and haven't been able to really consistently reproduce this.

@wesm do you have a pretty reliable set of actions that causes you to see this?

@wesm
Copy link
Contributor Author

wesm commented Feb 16, 2024

Yeah, if I close Positron, and then relaunch it from VSCode, I get it every time

@wesm
Copy link
Contributor Author

wesm commented Feb 16, 2024

Screen.Recording.2024-02-15.at.6.31.14.PM.mov

@wesm wesm added this to the Public Beta 2024 Q2 milestone Feb 20, 2024
@wesm
Copy link
Contributor Author

wesm commented Feb 21, 2024

This pop up is originating from this file:

https://github.com/microsoft/vscode/blob/main/src/vs/platform/encryption/electron-main/encryptionMainService.ts

the exact line that causes the pop-up to show up is

https://github.com/microsoft/vscode/blob/main/src/vs/platform/encryption/electron-main/encryptionMainService.ts#L64

	isEncryptionAvailable(): Promise<boolean> {
		return Promise.resolve(safeStorage.isEncryptionAvailable());
	}

See https://www.electronjs.org/docs/latest/api/safe-storage

On my computer, I have a bunch of Electron apps that seem to use this API:

image

If this quirk seems to only apply to running Positron in development mode, maybe we can just fix the "code-oss-dev" piece (somehow?) to say "Positron" instead and recommend that people authorize the keychain access?

See also microsoft/vscode-discussions#748

@wesm wesm added the area: builds Issues related to Builds category. label Feb 29, 2024
@juliasilge juliasilge modified the milestones: Public Beta 2024 Q2, Future Apr 1, 2024
@sharon-wang
Copy link
Member

Release build instance of this issue: #5485

@jonvanausdeln
Copy link
Contributor

Moving back into triage as we have at least 2 instances of this this in a release build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: builds Issues related to Builds category.
Projects
None yet
Development

No branches or pull requests

4 participants