Skip to content

Normalize trailing slash in store URLs - #59

Merged
omandryk merged 3 commits into
masterfrom
integration/pr-32
Aug 11, 2026
Merged

Normalize trailing slash in store URLs#59
omandryk merged 3 commits into
masterfrom
integration/pr-32

Conversation

@omandryk

Copy link
Copy Markdown
Owner

Summary

  • integrate the trailing-slash normalization contributed by @ekonopliov in Fixed URL formatting issue #32
  • preserve the contributor's original commit and authorship
  • add focused regression coverage for URLs with and without a trailing slash

Why

A store URL ending in / currently produces a double slash when the API path is appended. The contributor fix removes exactly one trailing slash during OAuthConfig construction.

Compatibility

  • no public API or binary signature changes
  • no changes to OAuth signature generation
  • URLs without a trailing slash are unchanged

Verification

  • mvn clean verify -Dgpg.skip=true
  • 9 tests, 0 failures, 0 errors, 7 existing live-store tests skipped
  • JAR, sources JAR, and Javadocs JAR built successfully

Credits @ekonopliov for the original fix in #32.

@omandryk
omandryk merged commit 6b6f123 into master Aug 11, 2026
4 checks passed
@omandryk
omandryk deleted the integration/pr-32 branch August 11, 2026 15:38
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.

1 participant