Skip to content

keyfix#21

Merged
nbassler merged 1 commit into
mainfrom
11-14
Apr 26, 2026
Merged

keyfix#21
nbassler merged 1 commit into
mainfrom
11-14

Conversation

@nbassler
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 26, 2026 12:08
@nbassler nbassler merged commit adfad53 into main Apr 26, 2026
2 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates Android release signing configuration to allow specifying a separate key password via environment variable.

Changes:

  • Use KEY_PASSWORD (with fallback to KEYSTORE_PASSWORD) for signingConfigs.release.keyPassword.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/build.gradle
storePassword = System.getenv("KEYSTORE_PASSWORD") ?: ""
keyAlias = System.getenv("KEY_ALIAS") ?: "dedx-upload"
keyPassword = System.getenv("KEYSTORE_PASSWORD") ?: ""
keyPassword = System.getenv("KEY_PASSWORD") ?: System.getenv("KEYSTORE_PASSWORD") ?: ""
Copy link

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The build now supports a separate key password via KEY_PASSWORD, but the release GitHub Actions workflow currently only exports KEYSTORE_PASSWORD (see .github/workflows/release.yml:38-42). If the key password differs from the keystore password, the CI release signing will still fail because it will fall back to KEYSTORE_PASSWORD. Consider updating the release workflow/secrets (and any docs) to set KEY_PASSWORD when needed, or explicitly document that both passwords must match.

Suggested change
keyPassword = System.getenv("KEY_PASSWORD") ?: System.getenv("KEYSTORE_PASSWORD") ?: ""
keyPassword = System.getenv("KEY_PASSWORD") ?: ""

Copilot uses AI. Check for mistakes.
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.

2 participants