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.

Sign up for GitHub

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

Jump to bottom

Feat/sqlite support #43

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Feat/sqlite support #43

wants to merge 3 commits into from

Conversation

jeyren95
Copy link
Contributor

@jeyren95 jeyren95 commented Mar 24, 2024 •
edited
Loading

Resolves #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
  • Migration scripts working
  • Sign up route
  • Sign in route
  • Update password route
  1. Postgres docker
  • Sign up route
  • Sign in route
  • Update password route
  1. Sqlite docker
  • Sign up route
  • Sign in route
  • Update password route

@jeyren95 jeyren95 mentioned this pull request Mar 31, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat: support sqlite
1 participant
@jeyren95