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.

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

Connection object has no attribute '_process_auth' #265

Closed
AlexLisovoy opened this issue Mar 7, 2018 · 5 comments
Closed

Connection object has no attribute '_process_auth' #265

AlexLisovoy opened this issue Mar 7, 2018 · 5 comments

Comments

@AlexLisovoy
Copy link
Contributor

Python 3.6, aiomysql 0.0.12, PyMySQL 0.8.0.

During connection to mysql server(5.7) I got following error:

...
File "./python3.6/site-packages/aiomysql/connection.py", line 712, in _request_authentication
    auth_packet = self._process_auth(plugin_name, auth_packet)
AttributeError: 'Connection' object has no attribute '_process_auth'

The above exception was the direct cause of the following exception:
...
  File "./python3.6/site-packages/aiomysql/connection.py", line 495, in _connect
    self._host) from e
pymysql.err.OperationalError: (2003, "Can't connect to MySQL server on '*****'")

Reading the code I didn't find implementation _process_auth at all.
AlexLisovoy added a commit to AlexLisovoy/aiomysql that referenced this issue Mar 7, 2018
@AlexLisovoy
Issue aio-libs#265 - _process_auth implementation
cfe838c
@AlexLisovoy
Copy link
Contributor Author

Here is #266 a pymysql-based implementation of this method. Please check and if it's fine then I will continue to work for adding tests.

@jettify
Copy link
Member

jettify commented Mar 7, 2018

Looks goo to me!

AlexLisovoy added a commit to AlexLisovoy/aiomysql that referenced this issue Mar 16, 2018
@AlexLisovoy
Issue aio-libs#265 - _process_auth implementation
61a039b
@younggyuchun
Copy link

Python 3.6, aiomysql 0.0.12, PyMySQL 0.8.0, mysql server(5.7.2.0) on MS Azure

I got the same error message.

However, connecting to '10.1.28-MariaDB' is totally fine.

jettify pushed a commit that referenced this issue Apr 19, 2018
@jettify
SSL Support (plus mysql_clear_password plugin for RDS) (#280)
dd068f4 
* Added SSL Support again

* Issue #265 - _process_auth implementation

* Added cleartext plugin test
@terricain
Copy link
Collaborator

@AlexLisovoy this should be fine now right?

@AlexLisovoy
Copy link
Contributor Author

@terrycain yep, it's good now.

@jettify jettify closed this as completed Apr 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jettify @AlexLisovoy @younggyuchun @terricain