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.

Sign up for GitHub

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

Jump to bottom

fix: do not ignore error of `url.Parse` #426

Merged

rueian merged 1 commit into redis:main from j178:main

Dec 16, 2023

Contributor

j178 commented Dec 15, 2023

If the error is ignored, there will be a nil pinter dereference panic when accessing u.Scheme.


          fix: do not ignore url.Parse error

509e07a

Collaborator

rueian commented Dec 16, 2023

Hi @j178, good catch and thank you for the test case!

rueian merged commit 8124203 into redis:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet