fix(pairing-cli): install rustls crypto provider - #2839
Open
bhargavms wants to merge 2 commits into
Open
Conversation
Signed-off-by: bhargavms <bhargav.m@ewa-services.com>
bhargavms
force-pushed
the
agent/fix-pairing-cli-rustls
branch
from
July 25, 2026 13:07
71eb8a2 to
05efbbb
Compare
bhargavms
marked this pull request as ready for review
July 25, 2026 13:15
Co-authored-by: Phuoc (Phu) Do <91568955+dophsquare@users.noreply.github.com> Signed-off-by: Phuoc (Phu) Do <91568955+dophsquare@users.noreply.github.com> * origin/main: (111 commits) chore(ci): bump desktop smoke E2E timeout to 30 minutes (block#3409) release(chart): publish 0.1.7 (block#3393) feat(acp): steer claude-code and codex agents via _session/steering (block#3007) feat(desktop): apply WebKit rendering workarounds at startup on Linux (block#3271) fix(desktop): stabilize flaky DM expansion E2E ordering assertions (block#2004) docs(contributing): document the Linux system libraries just ci requires (block#3396) fix(desktop): paint community rail full height (block#3382) fix(acp): disable goose cron scheduler in managed agent children (block#3144) feat(desktop): add custom harness inline from agent dialogs (block#3252) chore(compose): remove stale typesense env vars (block#3332) feat(desktop): refine agent catalog sharing (block#2439) fix(desktop): keep drafts out of the Inbox All view (block#3217) docs: restructure DCO guidance into scannable subsection (block#3337) Unify mobile loading spinners (block#3314) fix(desktop): restore the inbox icon in the sidebar (block#3341) fix(desktop): gate codex-acp on a minimum supported version (block#3254) feat(cli): add users set-status command for NIP-38 profile status (block#3253) fix(composer): scope multiline block formatting (block#3246) feat(chart): add relay pod extension points (block#3322) Refine mobile attachment picking (block#3313) ...
dophsquare
approved these changes
Jul 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ringrustls crypto provider before the pairing CLI opens a WSS connectionrustlsdependency so provider selection is stable under workspace feature unificationRoot cause
Workspace builds can enable both the
ringandaws-lc-rsrustls providers. When both are compiled, rustls cannot choose a process-level provider automatically, sobuzz-pair sourceandbuzz-pair targetpanic before connecting over WSS.Other Buzz binaries already select a provider explicitly; this applies the same pattern to
buzz-pair.Validation
just cicargo test -p buzz-pairing-clicargo clippy -p buzz-pairing-cli --all-targets -- -D warningsNo practical unit-test seam reproduces Cargo's workspace-wide feature unification inside the pairing CLI package alone, so the runtime regression was validated with the complete WSS transfer.