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

fix(lib): bug redis-go-cluster cannot convert a string type #141

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

tasszz2k
Copy link

Description

  • I tried to run Redis Full Check with this configuration
-s "10.50.69.13:30264;10.50.69.16:32592;10.50.69.15:31557" --sourcepassword z2ER6yw0hqUBjS5p --sourcedbtype 1 -t "10.50.69.13:30528;10.50.69.13:31136;10.50.69.16:31023" --targetdbtype 1 --targetpassword FK1ZMe43ih9pQWdI --sourcedbfilterlist="0" --targetdbfilterlist="0" --comparemode 1 --loglevel debug
  • Then I got an error redigo: Scan cannot convert from string to *string. Full log here:
main_go -s 10.50.69.13:30264;10.50.69.16:32592;10.50.69.15:31557 --sourcepassword z2ER6yw0hqUBjS5p --sourcedbtype 1 -t 10.50.69.13:30528;10.50.69.13:31136;10.50.69.16:31023 --targetdbtype 1 --targetpassword FK1ZMe43ih9pQWdI --sourcedbfilterlist=0 --targetdbfilterlist=0 --comparemode 1 --loglevel debug
[INFO 2024-08-21-17:25:35 main.go:65]: init log success
[INFO 2024-08-21-17:25:35 main.go:168]: configuration: {10.50.69.13:30264;10.50.69.16:32592;10.50.69.15:31557 z2ER6yw0hqUBjS5p auth 1 0 10.50.69.13:30528;10.50.69.13:31136;10.50.69.16:31023 FK1ZMe43ih9pQWdI auth 1 0 result.db  3 1 unknown unknown unknown 15000 5 256 5  debug false 16384  20445 false}
[INFO 2024-08-21-17:25:35 main.go:170]: ---------
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x20 pc=0x102d36eb4]

  • The root cause is the redis-go-cluster lib cannot convert string type to *string type with input is the redis enpoints (10.50.69.13:30264;10.50.69.16:32592;10.50.69.15:31557)

Solution

Signed-off-by: anhdt9 <anhdt9@vng.com.vn>
@CLAassistant
Copy link

CLAassistant commented Aug 21, 2024

CLA assistant check
All committers have signed the CLA.

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