Skip to content

Missing plugin name leads to panic #806

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

Closed
ios116 opened this issue May 24, 2018 · 6 comments · Fixed by #807
Closed

Missing plugin name leads to panic #806

ios116 opened this issue May 24, 2018 · 6 comments · Fixed by #807
Assignees
Labels
Milestone

Comments

@ios116
Copy link

ios116 commented May 24, 2018

Issue description

I would like to connect to sphinx search. I used to use this driver, but after the update, I receive an error.

Example code

db, err := sql.Open("mysql", "tcp(sphinx:9306)/")

Error log

unknown authentication plugin name ''

How can I solve this problem?

@julienschmidt
Copy link
Member

#807 might fix that issue

@julienschmidt julienschmidt added this to the v1.4.0 milestone May 24, 2018
@julienschmidt julienschmidt self-assigned this May 24, 2018
@collinalexbell
Copy link

I am getting the same error

@collinalexbell
Copy link

collinalexbell commented May 27, 2018

admin:somepass@tcp(someserver16:3306)/rgmsql?parseTime=true

This is the string I am using.

I reverted the commit made 8 days ago and got the slice bounds error that caused us to need #801

@julienschmidt
Copy link
Member

#807 should fix that. Please try again with the auth_refactoring branch.

@julienschmidt
Copy link
Member

Can one of you send us a packet dump of readInitPacket that we can use for a regression test? Just add a fmt.Println(data) after this line:

data, err := mc.readPacket()

@julienschmidt julienschmidt changed the title connect with sphinxsearch Missing plugin name leads to panic May 29, 2018
@julienschmidt
Copy link
Member

@fifsky provided a packet dump in #810

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

Successfully merging a pull request may close this issue.

3 participants