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.

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

make erun local warning bcrypt #314

Closed
robomotic opened this issue Oct 20, 2024 · 3 comments
Closed

make erun local warning bcrypt #314

robomotic opened this issue Oct 20, 2024 · 3 comments

Comments

@robomotic
Copy link

Nice project,
when I do: make run-local:

INFO:     Will watch for changes in these directories: ['/home/robomotic/DevOps/github/fastapi-nano']
INFO:     Uvicorn running on http://127.0.0.1:5002 (Press CTRL+C to quit)
INFO:     Started reloader process [32868] using StatReload
INFO:root:Running fastapi-nano with Python 3.13.0 (main, Oct 16 2024, 03:23:02) [Clang 18.1.8 ]
WARNING:passlib.handlers.bcrypt:(trapped) error reading bcrypt version
Traceback (most recent call last):
  File "/home/robomotic/DevOps/github/fastapi-nano/.venv/lib/python3.13/site-packages/passlib/handlers/bcrypt.py", line 620, in _load_backend_mixin
    version = _bcrypt.__about__.__version__
              ^^^^^^^^^^^^^^^^^
AttributeError: module 'bcrypt' has no attribute '__about__'
INFO:     Started server process [32870]
INFO:     Waiting for application startup.
INFO:     Application startup complete.

I think is just an issue with that version.

@rednafi
Copy link
Owner

rednafi commented Oct 21, 2024

Thanks for reporting. Yeah, I noticed this. It's an upstream issue that I'll need to take a look.

@priamai
Copy link

priamai commented Oct 22, 2024

Temporary fix in the svc/init.py:


import logging
import sys

# Set default logging value to debug
logging.basicConfig(level=logging.INFO)
# disabling due to: https://github.com/pyca/bcrypt/issues/684
logging.getLogger('passlib').setLevel(logging.ERROR)

logging.info("Running service with Python %s", sys.version)

@rednafi
Copy link
Owner

rednafi commented Oct 25, 2024

This has been patched #316

@rednafi rednafi closed this as completed Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants