• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

tari-project / tari / 16294860984
62%

Build:
DEFAULT BRANCH: development
Ran 15 Jul 2025 02:59PM UTC
Jobs 1
Files 723
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

15 Jul 2025 01:38PM UTC coverage: 57.833% (-0.02%) from 57.853%
16294860984

push

github

web-flow
fix: blocking main tokio thread when reading last latency (#7320)

Description
---
Fix minotari_console_wallet panic when attempting to block main tokio
thread with error message:
 ```
Cannot block the current thread from within a runtime. This happens
because a function attempted to block the current thread while the
thread is being used to drive asynchronous tasks
 ```
This happens because function `is_online` which was not async tried to
do blocking read on the `tokio::sync::RwLock` on main tokio thread. This
is not allowed since it would also block every other async action
causing in thread panic.

Motivation and Context
---
Fix sporadic wallet failures when repeatedly called `is_online` mostly
seen when bridging tokens.

How Has This Been Tested?
---
Just ran `cargo check` and verified that changing all functions
definition to `async` causes not error even so other functions in the
same trait were also `async`.

What process can a PR reviewer use to test or verify this change?
---
Basic code review of changes.

<!-- Checklist -->
<!-- 1. Is the title of your PR in the form that would make nice release
notes? The title, excluding the conventional commit
tag, will be included exactly as is in the CHANGELOG, so please think
about it carefully. -->


Breaking Changes
---

- [x] None
- [ ] Requires data directory on base node to be deleted
- [ ] Requires hard fork
- [ ] Other - Please specify

<!-- Does this include a breaking change? If so, include this line as a
footer -->
<!-- BREAKING CHANGE: Description what the user should do, e.g. delete a
database, resync the chain -->


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Refactor**
* Improved wallet connectivity checks to use asynchronous operations for
better performance and responsiveness. No user-facing changes in
functionality.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

0 of 5 new or added lines in 2 files covered. (0.0%)

34 existing lines in 5 files now uncovered.

69801 of 120694 relevant lines covered (57.83%)

225660.84 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
2
0.0
0.0% base_layer/wallet/src/connectivity_service/handle.rs
3
0.0
0.0% clients/rust/base_node_wallet_client/src/client/http.rs

Uncovered Existing Lines

Lines Coverage ∆ File
1
0.0
0.0% base_layer/wallet/src/connectivity_service/handle.rs
4
75.98
-0.87% comms/core/src/connection_manager/peer_connection.rs
4
85.22
-0.81% comms/core/src/noise/socket.rs
5
55.33
-0.54% comms/core/src/connectivity/manager.rs
20
60.37
-2.14% comms/dht/src/connectivity/mod.rs
Jobs
ID Job ID Ran Files Coverage
1 16294860984.1 15 Jul 2025 02:59PM UTC 723
57.83
GitHub Action Run
Source Files on build 16294860984
  • Tree
  • List 723
  • Changed 10
  • Source Changed 3
  • Coverage Changed 8
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #16294860984
  • 4b8745ed on github
  • Prev Build on development (#16287090853)
  • Next Build on development (#16313842248)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc