Skip to content

fix: normalize metrics, features and registration urls - #397

Open
sighphyre wants to merge 4 commits into
mainfrom
fix/normalize-urls
Open

fix: normalize metrics, features and registration urls#397
sighphyre wants to merge 4 commits into
mainfrom
fix/normalize-urls

Conversation

@sighphyre

Copy link
Copy Markdown
Member

Fixes an issue in the URL generation code for metrics, registration and features by preventing double slashes in the URL path when the user sets a Unleash server path with a trailing slash

@sighphyre
sighphyre force-pushed the fix/normalize-urls branch from 4816151 to c3a9e16 Compare July 24, 2026 07:54
@coveralls

coveralls commented Jul 24, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 30083961021

Coverage increased (+0.04%) to 93.238%

Details

  • Coverage increased (+0.04%) from the base build.
  • Patch coverage: 6 of 6 lines across 4 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 769
Covered Lines: 717
Line Coverage: 93.24%
Coverage Strength: 0.93 hits per line

💛 - Coveralls

@gastonfournier gastonfournier left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@github-project-automation github-project-automation Bot moved this from New to Approved PRs in Issues and PRs Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Approved PRs

Development

Successfully merging this pull request may close these issues.

4 participants