-
Notifications
You must be signed in to change notification settings - Fork 449
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
Extend password_hash
limit from User data model
#1137
Comments
I would suggest setting a postgres SQL database, which is what we are using in the deployed server. If any doubts you can ping me or @anitab-org/mentorship-maintainers and on Zulip => https://anitab-org.zulipchat.com/ 🤗 |
password_hash
limit from User data modelpassword_hash
limit from User data model
@isabelcosta I am a first-timer. I am interested in solving this issue, can you please assign it to me. :) |
Assigning you @mridubhatnagar |
@vj-codes @isabelcosta I have set up the project on my local system. Just wanted to understand how can I set up a database for the project? |
It should automatically create a sqllite database. |
On my local system when I run the server. I am getting an error sqlalchemy.exc.NoSuchModuleError: Can't load plugin: sqlalchemy.dialects:None One of the possible reason the above errors comes is due to spelling mistake in the database name sqllite instead sqlite. I did cross check config.py to make sure spellings are correct. I could not find any mistake in file though. Not sure what could be other possible reason for the error. |
Are you using docker or venv? |
venv |
@mridubhatnagar I have linked the possible fixes on zulip:) |
@vj-codes responded to you on zulip. :) |
Description
As a developer,
I need to remove the limit for password_hash,
so that I can fix the registration endpoint, that is erroring 500 INTERNAL SERVER ERROR, because of the limit being reached.
This relates to fixing #1109
A comment that may help: #1109 (comment)
Mocks
Acceptance Criteria
Update [Required]
password_hash
character limitDefinition of Done
Estimation
4h
The text was updated successfully, but these errors were encountered: