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

'Prepare Release' command is impractical to use with long keystore passwords #219

Open
davidgeary opened this issue Nov 25, 2024 · 2 comments · May be fixed by #221
Open

'Prepare Release' command is impractical to use with long keystore passwords #219

davidgeary opened this issue Nov 25, 2024 · 2 comments · May be fixed by #221

Comments

@davidgeary
Copy link

I've tried to use the 'Prepare Release' command but there doesn't seem to be a way to complete the process if both your keystore and key are password-protected with passwords that are long (i.e. secure) enough that you need to use copy/paste.

The issue is that the window prompting for the passwords is dismissed if you click away from it. This is avoidable when it's prompting for the keystore password as you can copy the password to the clipboard first, then start the command and paste the password when requested. But if your key also has a long password, as soon as you navigate to, say, your password manager to copy it, the window disappears, cancelling the command.

vscode-cmd-window

Obviously, you can get around this by typing the command, rather than copy-n-pasting, but this would discourage the use of long, secure passwords.

Is it possible to force the window to stay open or is this a restriction of the VS Code UI/extension API? (Or am I missing some other trick?)

This was referenced Nov 26, 2024
@dtarnawsky
Copy link
Collaborator

Hey @davidgeary , I've found a potential fix for this that prevents the dialog from closing when it loses focus. This is implemented in v1.99 which will come out soon (it's in the same batch as the changes for the migration to Angular 19 which are waiting on a release of angular-eslint).

@davidgeary
Copy link
Author

@dtarnawsky That's great! Thanks for looking into this.

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 a pull request may close this issue.

2 participants