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

Request to support more commands #4

Open
feihu-stripe opened this issue Aug 1, 2023 · 3 comments
Open

Request to support more commands #4

feihu-stripe opened this issue Aug 1, 2023 · 3 comments

Comments

@feihu-stripe
Copy link
Contributor

Request to support all the following commands:

mcp.CMD_ADD
mcp.CMD_SET
mcp.CMD_DELETE
mcp.CMD_GET
mcp.CMD_GETS
mcp.CMD_TOUCH
mcp.CMD_MA
mcp.CMD_MS
mcp.CMD_MD
mcp.CMD_MG
mcp.CMD_ME

This is what's supported today:

{"get", mcslib_get},
{"set", mcslib_set},
{"touch", mcslib_touch},
{"delete", mcslib_delete},
{"mg", mcslib_mg},
{"ms", mcslib_ms},
{"md", mcslib_md},
{"ma", mcslib_ma},
{"mg_factory", mcslib_mg_factory},
{"md_factory", mcslib_md_factory},
{"ma_factory", mcslib_ma_factory},
{"ms_factory", mcslib_ms_factory},

The diff is:

mcp.CMD_ADD
mcp.CMD_GETS
mcp.CMD_ME
@dormando
Copy link
Member

dormando commented Aug 1, 2023

Thanks!

The ME command is supposed to be a non-production-facing debug command. I wasn't intending on supporting that in mcshredder. Parsing the response is a lot too. Is that one necessary?

@feihu-stripe
Copy link
Contributor Author

You are right, I confirmed that ME is not used and should be removed from the list.

What's left are:

mcp.CMD_ADD
mcp.CMD_GETS

@feihu-stripe
Copy link
Contributor Author

Thank you!

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

No branches or pull requests

2 participants