Skip to content

Fix kv2 read version params#110

Draft
Varjoissa wants to merge 5 commits into
shahradelahi:mainfrom
Varjoissa:fix-kv2-read-version-params
Draft

Fix kv2 read version params#110
Varjoissa wants to merge 5 commits into
shahradelahi:mainfrom
Varjoissa:fix-kv2-read-version-params

Conversation

@Varjoissa

Copy link
Copy Markdown

Fixes #109

@changeset-bot

changeset-bot Bot commented May 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 33feac4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@litehex/node-vault Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Varjoissa Varjoissa marked this pull request as draft May 23, 2026 07:11
@shahradelahi shahradelahi self-requested a review May 29, 2026 17:11

@shahradelahi shahradelahi left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Could you please add unit tests?

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.

kv2.read with version parameter corrupts the URL path (Mustache HTML-escapes slashes)

2 participants