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

fix: signingConfigs for test apk #918

Merged
merged 1 commit into from
Sep 27, 2023
Merged

Conversation

fr0l
Copy link
Contributor

@fr0l fr0l commented Sep 26, 2023

Fixes issue when passing -PappiumKeystoreFile=/storeFile -PappiumKeystorePassword=passwd1 ... did not result in properly signed binary. It was always signed by AndroidDebugKey

Root cause of the issue - incorrect use of apply API in place, and creating File instance for passwords and alias.

How to verify:

./gradlew \
      signingReport

vs:

./gradlew \
      -PappiumZipAlign=true \
      -PappiumKeystoreFile="${PWD}/my_ci.keystore" \
      -PappiumKeystorePassword=my_ci_store_password \
      -PappiumKeyAlias=my_alias \
      -PappiumKeyPassword=my_ci_key_password \
      -PappiumTargetPackage=my.package \
      signingReport

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 26, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: fr0l / name: Vyacheslav Frolov (32a94c2)

@KazuCocoa KazuCocoa merged commit fcdd29f into appium:master Sep 27, 2023
11 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 27, 2023
## [2.25.9](v2.25.8...v2.25.9) (2023-09-27)

### Bug Fixes

* signingConfigs for test apk ([#918](#918)) ([fcdd29f](fcdd29f))
@github-actions
Copy link

🎉 This PR is included in version 2.25.9 🎉

The release is available on:

Your semantic-release bot 📦🚀

fr0l added a commit to fr0l/appium-espresso-driver that referenced this pull request Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants