Skip to content

fix(ci): use workflow_call hyphen secret names in java-release - #115

Merged
utkrishtsahu merged 1 commit into
masterfrom
fix/release-secret-refs
Jul 27, 2026
Merged

fix(ci): use workflow_call hyphen secret names in java-release#115
utkrishtsahu merged 1 commit into
masterfrom
fix/release-secret-refs

Conversation

@utkrishtsahu

Copy link
Copy Markdown
Contributor

Changes

Fixes the reusable release workflow so maven-publish reads the correct secret names.

.github/workflows/java-release.yml referenced secrets.OSSR_USERNAME, secrets.SIGNING_KEY, etc., but inside a workflow_call workflow only the hyphenated names declared in its secrets: block are in scope. The uppercase refs resolved to empty, so SIGNING_KEY/MAVEN_* reached Gradle blank and the 2.0.3 release failed at the signing {} block (nothing published, no tag created). Changed them to secrets.ossr-username, secrets.signing-key, etc., matching auth0/Auth0.Android.

CI/release wiring only — no library, API, or UI changes.

Testing

  • Static: the secrets.* refs now match the workflow_call: secrets: declarations in the same file.
  • Post-merge: re-run the release workflow and confirm publishToSonatype/closeSonatypeStagingRepository succeed with a staged com.auth0.android:jwtdecode:2.0.3 in the Central Portal.
  • No unit tests apply (workflow-only change); existing Build and Test CI runs unaffected.

[x] This change adds test coverage

[x] This change has been tested on the latest version of the platform/language or why not

Checklist

[x] I have read the Auth0 general contribution guidelines

[x] I have read the Auth0 Code of Conduct

[x] All existing and new tests complete without errors

@utkrishtsahu
utkrishtsahu requested a review from a team as a code owner July 27, 2026 06:34
@utkrishtsahu
utkrishtsahu merged commit 7f9abe4 into master Jul 27, 2026
4 checks passed
@utkrishtsahu
utkrishtsahu deleted the fix/release-secret-refs branch July 27, 2026 06:37
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