2
57%
master: 55%

Ran 05 Dec 2024 08:10AM UTC

Files 15

Run time 0s

Badge

Embed ▾

Committed 05 Dec 2024 07:01AM UTC coverage: 86.497%. Remained the same

Job # 12174808993.2

Build Type

Pull #2269

github

Committed by

same-id

Commit Message

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 Pull Request #2269: Fix non-root hosts failing on resolving DNS

Run Details

1531 of 1770 relevant lines covered (86.5%)

6439.38 hits per line

btcsuite / btcd / 12174808993 / 2
57%
master: 55%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job btcec - 12174808993.2

btcsuite / btcd / 12174808993 / 2 57% master: 55%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job btcec - 12174808993.2

btcsuite / btcd / 12174808993 / 2
57%
master: 55%

README BADGES
x