Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ntvviktor
Copy link

Attempt to split the order of private setStatus() function to allow sendCommand with setting client info metadata before Redis client would execute other users' command.

Try to resolve this issue #2025

@jit-ci
Copy link

jit-ci bot commented Oct 17, 2025

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

@ntvviktor
Copy link
Author

Hi @PavelPashov, let me know if this PR can help with the issue

@PavelPashov
Copy link
Contributor

Hi @PavelPashov, let me know if this PR can help with the issue

Hi @ntvviktor, thanks for your help with this issue, however it looks like the change has affected the CI. I’m also exploring a few approaches to resolve it.

@ntvviktor
Copy link
Author

@PavelPashov Yep, seems like my solution will not work well and compatible with other tests, looking forward to seeing yours, the underlining command executed and events replied from redis server is more complicated than it looks

@PavelPashov
Copy link
Contributor

Hey @ntvviktor, feel free to take a peek at #2033, however please note that it might change, let me know if you have any suggestions

@ntvviktor
Copy link
Author

Hey @PavelPashov, nice catch, btw I just wonder is this due to the root cause that Dragonfly DB is so fast and multi-thread that current approach of executing commands and replies would have problems?

@PavelPashov
Copy link
Contributor

Hey @PavelPashov, nice catch, btw I just wonder is this due to the root cause that Dragonfly DB is so fast and multi-thread that current approach of executing commands and replies would have problems?

I’m not able to confirm on whether that’s the underlying cause, but it’s an interesting hypothesis.

@ntvviktor ntvviktor closed this Oct 25, 2025
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.

2 participants