upstat icon indicating copy to clipboard operation
upstat copied to clipboard

Feat/sqlite support

Open jeyren95 opened this issue 1 year ago • 2 comments

Resolves https://github.com/chamanbravo/upstat/issues/34

Changes made:

  1. Database
  • removed pgcrypto from Postgres migration script
  • added separate migration scripts for Postgres and Sqlite
  1. 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
  1. Docker
  • updated Dockerfile to add sqlite support
  • added docker compose file for postgres users (default is sqlite)

Tested on following platforms:

  1. Localhost
  • [x] Migration scripts working
  • [x] Sign up route
  • [x] Sign in route
  • [x] Update password route
  1. Postgres docker
  • [x] Sign up route
  • [x] Sign in route
  • [x] Update password route
  1. Sqlite docker
  • [x] Sign up route
  • [x] Sign in route
  • [x] Update password route

jeyren95 avatar Mar 24 '24 14:03 jeyren95

@chamanbravo Can we merge this?

yonas avatar Aug 15 '24 15:08 yonas

hello @yonas, thank you for the remainder, I will try to merge this asap. Thank you @jeyren95 for your great work.

chamanbravo avatar Aug 17 '24 05:08 chamanbravo

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

jeyren95 avatar Aug 25 '24 14:08 jeyren95

Its okay and thank you so much for your contributions.

chamanbravo avatar Aug 25 '24 15:08 chamanbravo