Skip to content

Conversation

@mohitrajain
Copy link
Contributor

@mohitrajain mohitrajain commented Dec 18, 2025

Changed from Ansible variable slicing (which failed with binary data) to file-based operations: The fix replaces direct byte extraction from decoded DER data using Ansible's b64decode and string slicing with shell commands that write DER data to files and use tail -c 32 to extract the exact 32-byte chunks needed for Ed25519 keys.

closing - #837

Change type

  • Fix
  • Feature
  • Documentation
  • Security / Upgrade

Basic information

  • THIS CHANGE REQUIRES A DEPLOYMENT PACKAGE RELEASE
  • THIS CHANGE REQUIRES A WIRE-DOCS RELEASE

Testing

  • I ran/applied the changes myself, in a test environment.
  • The CI job attached to this repo will test it for me.

Offline Build CI (label-based)

Add one or more labels to trigger offline builds:

  • build-default - Full production build (ansible, terraform, all packages)
  • build-demo - Demo/WIAB build
  • build-min - Minimal build (fastest, essential charts only)
  • build-all - Run all three builds

Note: No builds run by default. Add a label to trigger CI.

Tracking

  • I added a new entry in an appropriate subdirectory of changelog.d
  • I mentioned this PR in Jira, OR I mentioned the Jira ticket in this PR.
  • I mentioned this PR in one of the issues attached to one of our repositories.

Knowledge Transfer

  • An Asciinema session is attached to the Jira ticket.

Motivation

Objective

Reason

Use case

@mohitrajain mohitrajain requested review from a team and julialongtin as code owners December 18, 2025 16:28
@mohitrajain mohitrajain changed the title Patch zauth fixes wpn22439 Patch zauth fixes wpb-22439 Dec 18, 2025
@sonarqubecloud
Copy link

@julialongtin
Copy link
Member

note: removing proc_pull_ext_charts.sh in this PR. not needed, due to recent improvements in wire-builds.

@mohitrajain mohitrajain merged commit 8efc126 into master Dec 19, 2025
13 of 15 checks passed
@mohitrajain mohitrajain deleted the patch-zauth-fixes-wpn22439 branch December 19, 2025 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants