Skip to content

Conversation

@mafredri
Copy link
Owner

@mafredri mafredri commented Dec 7, 2025

A trailing slash in the URL caused double-slash paths like "//json/version",
which failed with the misleading error "could not resolve IP for 127.0.0.1".

Now we strip trailing slashes and show the actual endpoints tried on failure.

Fixes #152

A trailing slash in the URL caused double-slash paths like "//json/version",
which failed with the misleading error "could not resolve IP for 127.0.0.1".

Now we strip trailing slashes and show the actual endpoints tried on failure.

Fixes #152
@codecov
Copy link

codecov bot commented Dec 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 39.7%. Comparing base (3b1f600) to head (688b926).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##            main    #155     +/-   ##
=======================================
+ Coverage   39.5%   39.7%   +0.2%     
=======================================
  Files         25      25             
  Lines       2322    2325      +3     
=======================================
+ Hits         918     924      +6     
+ Misses      1363    1362      -1     
+ Partials      41      39      -2     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mafredri mafredri merged commit db17eca into main Dec 7, 2025
7 checks passed
@mafredri mafredri deleted the fix-trailing-slash-url branch December 7, 2025 21:45
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.

Trailing slash in URL results in could not resolve IP for 127.0.0.1

2 participants