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

[BUG] - Warning Message on Password Change Within Docker #3040

Closed
5 of 6 tasks
firefly2442 opened this issue Jan 24, 2024 · 1 comment · Fixed by #3045
Closed
5 of 6 tasks

[BUG] - Warning Message on Password Change Within Docker #3040

firefly2442 opened this issue Jan 24, 2024 · 1 comment · Fixed by #3045
Labels
bug: confirmed bug Something isn't working

Comments

@firefly2442
Copy link
Contributor

firefly2442 commented Jan 24, 2024

First Check

  • This is not a feature request.
  • I added a very descriptive title to this issue (title field is above this).
  • I used the GitHub search to find a similar issue and didn't find it.
  • I searched the Mealie documentation, with the integrated search.
  • I already read the docs and didn't find an answer.
  • This issue can be replicated on the demo site (https://demo.mealie.io/).

What is the issue you are experiencing?

When I changed a password per the instructions in the FAQ, it was successful but did have a warning message about the bcrypt library.

Steps to Reproduce

username@myhost:~/src/mealie$ docker exec -it 1bb bash
root@1bb66895c272:/# python /app/mealie/scripts/change_password.py
Please enter the email of the user you want to reset: admin
INFO: 23-Jan-24 20:38:09 	changing password for admin
Please enter the new password: 
Please enter the new password again: 
WARNING: 23-Jan-24 20:38:13 	(trapped) error reading bcrypt version
Traceback (most recent call last):
  File "/opt/pysetup/.venv/lib/python3.10/site-packages/passlib/handlers/bcrypt.py", line 620, in _load_backend_mixin
    version = _bcrypt.__about__.__version__
AttributeError: module 'bcrypt' has no attribute '__about__'
INFO: 23-Jan-24 20:38:14 	password change successful
press enter to exit 

Please provide relevant logs

From docker compose logs:

mealie  | INFO:     192.168.1.124:57394 - "GET /api/app/about HTTP/1.1" 200 OK
mealie  | INFO:     192.168.1.124:47616 - "GET /api/app/about HTTP/1.1" 200 OK
mealie  | INFO:     192.168.1.124:47618 - "GET /api/app/about/startup-info HTTP/1.1" 200 OK
mealie  | INFO:     172.19.0.1:47098 - "GET / HTTP/1.1" 200 OK
mealie  | WARNING: 23-Jan-24 20:36:32 	(trapped) error reading bcrypt version
mealie  | Traceback (most recent call last):
mealie  |   File "/opt/pysetup/.venv/lib/python3.10/site-packages/passlib/handlers/bcrypt.py", line 620, in _load_backend_mixin
mealie  |     version = _bcrypt.__about__.__version__
mealie  | AttributeError: module 'bcrypt' has no attribute '__about__'

Mealie Version

ghcr.io/mealie-recipes/mealie:latest (v1.0.0 version)

Deployment

Docker (Linux)

Additional Deployment Details

Linux pifour 5.15.0-1044-raspi #47-Ubuntu SMP PREEMPT Tue Nov 21 11:33:46 UTC 2023 aarch64 aarch64 aarch64 GNU/Linux

Ubuntu 22.04 on Raspberry Pi 4b.

@firefly2442 firefly2442 added bug Something isn't working triage labels Jan 24, 2024
@michael-genson
Copy link
Collaborator

michael-genson commented Jan 24, 2024

This is a known issue with bcrypt/passlib. It's harmless, just a warning. We should, however, suppress that log if we can, so I'll leave this issue open

FYI: pyca/bcrypt#684

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: confirmed bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants