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

Apple Silicon: error with keychain #111271

Closed
ItalyPaleAle opened this issue Nov 25, 2020 · 5 comments
Closed

Apple Silicon: error with keychain #111271

ItalyPaleAle opened this issue Nov 25, 2020 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Milestone

Comments

@ItalyPaleAle
Copy link
Contributor

Version: 1.52.0-exploration
Commit: 88c7381
Date: 2020-11-20T09:15:28.497Z (4 days ago)
Electron: 11.0.2
Chrome: 87.0.4280.67
Node.js: 12.18.3
V8: 8.7.220.25-electron.0
OS: Darwin arm64 20.1.0

Steps to Reproduce:

  1. On a fresh install, enable settings sync (using the Stable channel)
  2. Sign in with a GitHub account
  3. After logging in on the browser, go back to VS Code Exploration
  4. The error below appears:
    image

Despite the error, settings to start syncing. And then the window crash multiple times... until it works! (probably connected to #111091?)

Does this issue occur when all extensions are disabled?: Fresh install w/ no extensions

@deepak1556
Copy link
Collaborator

Oh weird the keytar module is built for x64, looks like cross-compiling didn't work.

@deepak1556 deepak1556 added the bug Issue identified by VS Code Team member as probable bug label Nov 25, 2020
@deepak1556 deepak1556 self-assigned this Nov 25, 2020
@deepak1556
Copy link
Collaborator

I am currently using electron-prebuild to cross-compile native modules, and it looks like the package doesn't rebuild prebuilt modules from source https://github.com/electron/electron-rebuild/blob/master/src/module-rebuilder.ts#L296. Keytar is the only prebuilt module we have now, I will add a separate rebuild step for it.

@roblourens
Copy link
Member

@deepak you will probably have to verify this one yourself...

@ItalyPaleAle
Copy link
Contributor Author

@roblourens I can confirm that it is working on the latest Exploration build

@roblourens roblourens added the verified Verification succeeded label Dec 7, 2020
@roblourens
Copy link
Member

Thanks!

@github-actions github-actions bot locked and limited conversation to collaborators Jan 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants
@ItalyPaleAle @roblourens @deepak1556 @rzhao271 and others