flask-security icon indicating copy to clipboard operation
flask-security copied to clipboard

Published 20 hours ago • verified publisher icon mattupstate

SQLAlchemyUserDatastore.get_user should use get with uuid's

Open cancan101 opened this issue 6 years ago • 2 comments

Currently get is used only for numerics rather than also for a UUID pk: https://github.com/mattupstate/flask-security/blob/3e15d06ee82a728d5cc53059fb28406c7bc6e7aa/flask_security/datastore.py#L242-L243

cancan101 avatar Apr 23 '19 05:04 cancan101

I believe the #633 actually solves this. That being said - it appears there are numerous issues with simply trying different columns and looking for a match. Question - can you share your UserModel - did you change the PK from an Integer type to string? or did you just add a new field?

jwag956 avatar May 04 '19 02:05 jwag956

This has been fixed and merged in https://github.com/jwag956/flask-security in 3.1.0rc1 This is available at https://pypi.org/project/Flask-Security-Too/#history

jwag956 avatar May 28 '19 01:05 jwag956