Skip to content

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

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants