• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In
Warning: This build has drifted.
The coverage report for this pull request build may be inaccurate because its base commit is no longer the HEAD of its target branch.
This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

    • Learn more: For more information on this, see Tracking coverage changes for pull request builds.
    • Fix now: For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
    • Prevent going forward: To avoid this issue with future PRs, see these Recommended CI Configurations.
New Repo Setting:
INCLUDE COVERAGE % WITH WARNINGS ABOUT DRIFTED BUILDS?

Enabling this setting will include a (potentially inaccurate) coverage % with warning messages in status updates for drifted builds.

Adjust setting

btcsuite / btcd / 11554179775
57%
master: 55%

Build:
Build:
LAST BUILD BRANCH: spellcheck
DEFAULT BRANCH: master
Ran 06 Nov 2024 06:17PM UTC
Jobs 4
Files 237
Run time 1min
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

28 Oct 2024 12:42PM UTC coverage: 57.223% (+0.01%) from 57.213%
11554179775

Pull #2269

github

same-id
Fix non-root hosts failing on resolving DNS

A fix for a bug introduced by #2168

Previously, config.Host worked in the following way:
1. Documented as supporting ip addresses only
2. In fact supported "host/path" syntax
3. Did not support "scheme" prefixes, i.e. https://

Not sure this is the desired approach, probably the best thing would
have been to extend config to contain "Scheme" and "Path" fields as well.

However, this was the way it worked.

1. Now Host can contain scheme prefixes "unix://..."
2. Host can no longer contain ".../path"

This PR solves this behavior while maintaining support of the "unix://" flow
as well.

For some reason, "scheme" is named "network" in #2168 - I did not change that.

Also remove disambiguation in "network:address:port", where it parsed
"myhost:8888" as network:address instead address:port.
Pull Request #2269: Fix non-root hosts failing on resolving DNS

9 of 17 new or added lines in 1 file covered. (52.94%)

10 existing lines in 1 file now uncovered.

29849 of 52163 relevant lines covered (57.22%)

423037.75 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
8
38.74
-0.34% rpcclient/infrastructure.go

Uncovered Existing Lines

Lines Coverage ∆ File
10
38.74
-0.34% rpcclient/infrastructure.go
Jobs
ID Job ID Ran Files Coverage
1 btcec - 11554179775.1 06 Nov 2024 06:17PM UTC 15
86.5
GitHub Action Run
2 btcutil - 11554179775.2 06 Nov 2024 06:17PM UTC 21
87.22
GitHub Action Run
3 btcutilpsbt - 11554179775.3 06 Nov 2024 06:17PM UTC 13
76.48
GitHub Action Run
4 btcd - 11554179775.4 06 Nov 2024 06:17PM UTC 188
53.41
GitHub Action Run
Source Files on build 11554179775
  • Tree
  • List 237
  • Changed 59
  • Source Changed 0
  • Coverage Changed 3
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Pull Request #2269
  • PR Base - master (#11292783584)
  • Delete
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

© 2025 Coveralls, Inc