upstat
upstat copied to clipboard
Feat/sqlite support
Resolves https://github.com/chamanbravo/upstat/issues/34
Changes made:
- Database
- removed pgcrypto from Postgres migration script
- added separate migration scripts for Postgres and Sqlite
- API
- added sqlite connection as default
- used bcrypt to hash and salt passwords
- updated signing in, signing up and update password controllers to use bcrypt hashing
- Docker
- updated Dockerfile to add sqlite support
- added docker compose file for postgres users (default is sqlite)
Tested on following platforms:
- Localhost
- [x] Migration scripts working
- [x] Sign up route
- [x] Sign in route
- [x] Update password route
- Postgres docker
- [x] Sign up route
- [x] Sign in route
- [x] Update password route
- Sqlite docker
- [x] Sign up route
- [x] Sign in route
- [x] Update password route
@chamanbravo Can we merge this?
hello @yonas, thank you for the remainder, I will try to merge this asap. Thank you @jeyren95 for your great work.
thanks for helping to resolve the conflicts and merging to the main branch, was quite busy the past week so didnt have the time to look at it yet
Its okay and thank you so much for your contributions.