Skip to content

Make Checkout / Quick Checkout / Push for Review work without configuring git auth separately - #15

Open
albin-paul-saju wants to merge 7 commits into
mainfrom
fix-quickcheckout-password-issue
Open

Make Checkout / Quick Checkout / Push for Review work without configuring git auth separately#15
albin-paul-saju wants to merge 7 commits into
mainfrom
fix-quickcheckout-password-issue

Conversation

@albin-paul-saju

@albin-paul-saju albin-paul-saju commented Apr 27, 2026

Copy link
Copy Markdown
Collaborator

What

Make Checkout, Quick Checkout, and Push for Review use the credentials already entered via "Gerrit: Enter credentials". No separate git credential helper.

Why

Those credentials live in VS Code SecretStorage and are only used for Gerrit's REST API. When the extension shells out (e.g. git fetch origin refs/changes/35/328435/5), git looks them up from its own places (credential.helper, ~/.netrc, SSH) and, finding nothing on a stock machine, fails non-interactively with:

fatal: could not read Username for 'https://nugerrit.ntnxdpro.com':
       No such device or address

REST shows "Gerrit connection successful!" while every checkout/push silently breaks - the root cause of the "Quick checkout asks for password" report.

How

Per-invocation GIT_ASKPASS shim (src/lib/git/gerritAskpass.ts) that injects the stored Gerrit creds into the single git child process the extension is about to spawn - and nothing else. No global state, no secrets on disk.

@albin-paul-saju albin-paul-saju changed the title Quick checkout asks for password Make Checkout / Quick Checkout / Push for Review work without configuring git auth separately Apr 27, 2026
@albin-paul-saju
albin-paul-saju force-pushed the fix-invalid-change-id-bug branch from ecbc621 to 6486b06 Compare April 27, 2026 12:17
@albin-paul-saju
albin-paul-saju force-pushed the fix-quickcheckout-password-issue branch from 555cb30 to 139a4aa Compare April 27, 2026 12:24
Base automatically changed from fix-invalid-change-id-bug to main April 28, 2026 04:05
@albin-paul-saju
albin-paul-saju force-pushed the fix-quickcheckout-password-issue branch from 139a4aa to 95d4fb8 Compare April 28, 2026 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant