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

contrib/go-redis/redis.v8: fix broken parsing of redis commands #1783

Merged
merged 6 commits into from
May 12, 2023

Commits on Mar 6, 2023

  1. contrib/go-redis/redis.v8: fix broken parsing of redis commands

    The previous code made some assumptions about the format of the commands
    that were not correct, namely that they would always contain at least 1
    space. This seems to be true for commands generated by the library, but
    not necessarily for commands given as text to the Do function, e.g.
    client.Do(ctx, "PING").
    
    This results in a panic due to out-of-bounds indexing.
    
    This commit makes the parsing code not assume this format and adds a test
    case to ensure it works correctly for commands with zero spaces.
    knusbaum committed Mar 6, 2023
    Configuration menu
    Copy the full SHA
    990e5f5 View commit details
    Browse the repository at this point in the history

Commits on Apr 20, 2023

  1. Configuration menu
    Copy the full SHA
    e777770 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    0701a9a View commit details
    Browse the repository at this point in the history

Commits on May 10, 2023

  1. Configuration menu
    Copy the full SHA
    99dbc2c View commit details
    Browse the repository at this point in the history

Commits on May 12, 2023

  1. Configuration menu
    Copy the full SHA
    cb4afb7 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    c785de8 View commit details
    Browse the repository at this point in the history