Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 BUG: kv local list doesn't respect --prefix arg #3716

Closed
JacksonKearl opened this issue Aug 7, 2023 · 0 comments · Fixed by #3720
Closed

🐛 BUG: kv local list doesn't respect --prefix arg #3716

JacksonKearl opened this issue Aug 7, 2023 · 0 comments · Fixed by #3720
Assignees
Labels
bug Something that isn't working

Comments

@JacksonKearl
Copy link
Contributor

JacksonKearl commented Aug 7, 2023

Which Cloudflare product(s) does this pertain to?

KV, Wrangler core

What version(s) of the tool(s) are you using?

nightly

What version of Node are you using?

No response

What operating system are you using?

Mac

Describe the Bug

See here:

➜  Recipe git:(main) ✗ npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/5725468543/npm-package-wrangler-3684 kv:key list --local --binding STORE --preview --prefix rex
{
  "keys": [
    {
      "name": "anotherone",
      "expiration": 1693440000
    },
    {
      "name": "myNewKey"
    },
    {
      "name": "quota/token/toker",
      "metadata": "\"\\\"boopers\\\"\""
    },
    {
      "name": "rexipie/09z9dnu8rgezpi7rcmeelbrq"
    },

Please provide a link to a minimal reproduction

No response

Please provide any relevant error logs

No response

cc @jspspike

@JacksonKearl JacksonKearl added the bug Something that isn't working label Aug 7, 2023
JacksonKearl added a commit to JacksonKearl/cloudflare-vscode-devtools that referenced this issue Aug 7, 2023
@jspspike jspspike self-assigned this Aug 7, 2023
JacksonKearl added a commit to JacksonKearl/workers-sdk that referenced this issue Aug 8, 2023
jspspike pushed a commit that referenced this issue Aug 9, 2023
* Unify `kv:key list` output in local vs remote case
Fix #3715, Fix #3716

* Add tests
lrapoport-cf pushed a commit that referenced this issue Aug 11, 2023
* Unify `kv:key list` output in local vs remote case
Fix #3715, Fix #3716

* Add tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants