Skip to content

fix: add missing normalize calls to keys in various areas#2

Merged
ldez merged 2 commits intokvtools:mainfrom
ainesophaur:key-not-normalized
Sep 19, 2025
Merged

fix: add missing normalize calls to keys in various areas#2
ldez merged 2 commits intokvtools:mainfrom
ainesophaur:key-not-normalized

Conversation

@ainesophaur
Copy link
Contributor

@ainesophaur ainesophaur commented Mar 13, 2023

I noticed that keys prefixed with / have the prefix stripped throughout various methods of the adapter (get, delete, etc) but missing in some methods like put. This PR provides the intended behavior that keys set though the store are able to be retrieved when the key is prefixed with a /

traefik/traefik#11416

@ldez ldez added the bug Something isn't working label Mar 14, 2023
@ldez ldez self-requested a review September 19, 2025 01:52
@ldez
Copy link
Member

ldez commented Sep 19, 2025

@ainesophaur Sorry, I forgot your PR.
If you revert the unrelated changes, I will review and merge it.

@ainesophaur
Copy link
Contributor Author

I reset the changes and repushed. Let me know if it needs anything else.

@ldez ldez changed the title add missing normalize calls to keys in various areas fix: add missing normalize calls to keys in various areas Sep 19, 2025
Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

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

LGTM

@ldez ldez merged commit 95e3034 into kvtools:main Sep 19, 2025
2 checks passed
@ldez
Copy link
Member

ldez commented Sep 19, 2025

I created a new tag v1.0.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

2 participants