Skip to content

Conversation

@k80bowman
Copy link
Contributor

W-17704930

Resolves #3186

Sets the database arg to ignore stdin in order to allow a user to pipe commands through to their redis instance.

Testing

  1. Check out this branch and run yarn && yarn build
  2. Run heroku addons:create heroku-redis:mini -a TEST_APP to install a redis instance. Wait for the addon to be provisioned.
  3. Run echo "test" | ./bin/run redis:cli -a TEST_APP and verify that you see a message that the CLI is connecting to your redis instance, the word "test" is printed, and then you get an error about the "unknown command 'test'", and then the connection closes. You should not see an "Unexpected argument" error or a "No Redis instances found error and the connection should close and not remain open.

@k80bowman k80bowman requested a review from a team as a code owner September 12, 2025 18:52
@k80bowman k80bowman merged commit f6a4e3c into main Sep 12, 2025
8 checks passed
@k80bowman k80bowman deleted the k80/redis-cli-bug branch September 12, 2025 20:04
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

Successfully merging this pull request may close these issues.

Piping commands to Redis CLI fails

2 participants