Support for Python 3.12 about fastapi-users

[frankie567]

Fixed as of v13.0.0

[frankie567]

Thanks for raising this @davidbrochart 👍

Unfortunately, passlib seems to be unmaintained those days: https://foss.heptapod.net/python-libs/passlib

It's really unfortunate, because passlib is the de-facto standard for password hashing. I'm not aware of alternatives 🤔

[davidbrochart]

Maybe fork it?

[frankie567]

Probably, but not me. I know nothing about cryptography algorithms, so I wouldn't risk to support a library that critical 😅

[davidbrochart]

An alternative is to install setuptools, but there is a deprecation warning when importing:

Python 3.12.0 | packaged by conda-forge | (main, Oct  3 2023, 08:43:22) [GCC 12.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import pkg_resources
<stdin>:1: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html

[bert2002]

Not sure if its related, but just started seeing this:

Python: 3.12.0
FastAPI: 0.104.1
FastAPI-Users: 12.1.2

INFO:     172.21.0.1:47400 - "GET /auth/google/authorize HTTP/1.1" 200 OK
(trapped) error reading bcrypt version
Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/fastapi_users/manager.py", line 201, in oauth_callback
    user = await self.get_by_oauth_account(oauth_name, account_id)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/fastapi_users/manager.py", line 106, in get_by_oauth_account
    raise exceptions.UserNotExists()
fastapi_users.exceptions.UserNotExists
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/fastapi_users/manager.py", line 205, in oauth_callback
    user = await self.get_by_email(account_email)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/fastapi_users/manager.py", line 90, in get_by_email
    raise exceptions.UserNotExists()
fastapi_users.exceptions.UserNotExists
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/passlib/handlers/bcrypt.py", line 620, in _load_backend_mixin
    version = _bcrypt.__about__.__version__
              ^^^^^^^^^^^^^^^^^
AttributeError: module 'bcrypt' has no attribute '__about__'
User 4bdb7054-ddd9-433c-8bbb-a7204a296ae8 has registered.

This happens when registering a new user, it works perfectly and have not noticed any problems...

bert

[tcompa]

I think that the warning in the previous comment (by @bert2002) is unrelated to python 3.12; it's rather a passlib issue with recent bcrypt versions (see https://foss.heptapod.net/python-libs/passlib/-/issues/190).

---

Unfortunately, passlib seems to be unmaintained those days: https://foss.heptapod.net/python-libs/passlib

It's really unfortunate, because passlib is the de-facto standard for password hashing. I'm not aware of alternatives 🤔

For the record, there seems to be a renewed effort towards passlib maintenance, see https://foss.heptapod.net/python-libs/passlib/-/issues/187.

[gaetanDubuc]

The only dependency to passlib seems to be in password.py. https://argon2-cffi.readthedocs.io/en/stable/argon2.html could replace passlib as the package defines a PasswordHasher with close protocol. We just have to define a new CryptContext protocol.

[frankie567]

The only dependency to passlib seems to be in password.py. https://argon2-cffi.readthedocs.io/en/stable/argon2.html could replace passlib as the package defines a PasswordHasher with close protocol. We just have to define a new CryptContext protocol.

Passlib supports several hashing algorithms, not only argon2, and has convenient features like hash upgrades. Plus, I believe most FastAPI Users installations use bcrypt right now.

[JimScope]

Hi @frankie567 could this option to unblock the library for python 3.12 be considered as a short term solution? #1345

[MatthewScholefield]

A passlib maintainer responded so it sounds like there may be a fix coming soon.