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

Make INFO command reject cases with arguments greater than 2 #1570

Merged
merged 3 commits into from
Jul 10, 2023

Commits on Jul 9, 2023

  1. Made INFO command reject cases with arguments greater than 2

    Passing `info a b c` is the same as passing plain `info`,
    since we only checked that args_.size() == 2, by default it
    will output all.
    
    In this PR, we will reject args_.size() > 2 and return a
    syntax error. This is consistent with Redis 6.2 behavior,
    however, it is inconsistent with Redis 7.0, Redis 7.0 supports
    passing `info server clients ...`.
    
    Because we don't plan to do such support for the time being,
    we will first fix the parts that are obviously problematic.
    enjoy-binbin committed Jul 9, 2023
    Configuration menu
    Copy the full SHA
    a7e2349 View commit details
    Browse the repository at this point in the history

Commits on Jul 10, 2023

  1. Configuration menu
    Copy the full SHA
    002147a View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    55d8178 View commit details
    Browse the repository at this point in the history