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

Use sqlx inplace of rusqlite #242

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Commits on Sep 17, 2023

  1. abandon teos-common dbm

    mariocynicys committed Sep 17, 2023
    Configuration menu
    Copy the full SHA
    e37782d View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    8bbe5fa View commit details
    Browse the repository at this point in the history
  3. Upgrade to rusqlite 0.29

    This is temporary to avoid a dependency conflict that arises in `libsqlite3-sys`
    mariocynicys committed Sep 17, 2023
    Configuration menu
    Copy the full SHA
    60e3eb7 View commit details
    Browse the repository at this point in the history
  4. Implementing and using the sqlx drived DBM

    The two DBMs are now in use, this is an intermediary commit and `dbm.rs` should be deleted and replaced with `dbm_new.rs`, and the tower components should be adjusted accordingly (drop the mutex around the dbm since the pool handles it by itself & await on dbm methods since they are now async)
    mariocynicys committed Sep 17, 2023
    Configuration menu
    Copy the full SHA
    b5ba42b View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2023

  1. Moving the tower components to async

    Some mutexed needed their lifetimes be shortened due to the fact that an std mutex guard can't live across await point. one specific mutex on GateKeeper::registered_users was more fitting to be a tokio mutex instead due to the nature of how it is uses beside the database usage (std mutex is generally better/has less overhead than a tokio mutex).
    
    tests still need to be patched for async
    mariocynicys committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    9c9f7fd View commit details
    Browse the repository at this point in the history
  2. Adjusted the tests (basically adding .awaits)

    Postgres tests are still not working because with each test we need to spawn a brand new database (best done inside a docker container) from within the tests
    mariocynicys committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    38abec0 View commit details
    Browse the repository at this point in the history

Commits on Oct 16, 2023

  1. fixing lint errors

    mariocynicys committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    4bd2051 View commit details
    Browse the repository at this point in the history
  2. fix grammar issues

    mariocynicys committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    ba321f4 View commit details
    Browse the repository at this point in the history
  3. no need for type =

    mariocynicys committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    e5519d0 View commit details
    Browse the repository at this point in the history
  4. mysql todo

    mariocynicys committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    233b604 View commit details
    Browse the repository at this point in the history
  5. fix postgresql tests

    this still needs an external test db running on the background. one can easily be spawned with docker
    mariocynicys committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    178729f View commit details
    Browse the repository at this point in the history
  6. document clear_db method

    mariocynicys committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    d42d4bc View commit details
    Browse the repository at this point in the history
  7. remove DB logic from teos-common

    remove rusqlite dependency from teos-common
    mariocynicys committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    3aaa7c9 View commit details
    Browse the repository at this point in the history
  8. elaborating a comment

    mariocynicys committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    0d4e6ea View commit details
    Browse the repository at this point in the history
  9. fix a typo

    mariocynicys committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    782ed39 View commit details
    Browse the repository at this point in the history