Skip to content

fix: pin trezor safe7 transport#87

Closed
ovitrif wants to merge 1 commit intomasterfrom
fix/trezor-7-usb-2
Closed

fix: pin trezor safe7 transport#87
ovitrif wants to merge 1 commit intomasterfrom
fix/trezor-7-usb-2

Conversation

@ovitrif
Copy link
Copy Markdown
Collaborator

@ovitrif ovitrif commented Apr 9, 2026

Description

  • pin trezor-connect-rs to commit 214d0c542ad50992811df3a83ab3e1be20ec4cad from ovitrif/trezor-connect-rs
  • pull in the USB Safe 7 THP transport fix that reopens the device after the legacy V1 probe before starting THP
  • rebuild the Android native libraries so the published Gradle package uses the patched transport implementation

Preview

  • n/a

QA Notes

  • ran cargo test modules::trezor
  • ran cargo build
  • ran cargo clippy
  • ran ./build.sh android
  • ./build.sh android completed successfully, including publishToMavenLocal
  • full cargo test in this repo still hit unrelated blocktank test failures/hangs, so validation was focused on the Trezor path

@claude
Copy link
Copy Markdown

claude Bot commented Apr 9, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@ovitrif ovitrif marked this pull request as draft April 9, 2026 22:02
@ovitrif
Copy link
Copy Markdown
Collaborator Author

ovitrif commented Apr 28, 2026

didn't fix it, I retested on Android via local changes integrating this.

@ovitrif ovitrif closed this Apr 28, 2026
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.

1 participant