search

strager / First

GNU General Public License v3.0
3 stars 2 forks source link

Merge add_new_user and update_tokens in AuthDb #4

Closed strager closed 1 year ago

mate-amargo commented 1 year ago

So, merging this two functions will result in the removal of the check for a non existent user (removing this test), since the new function will just create a new user if it doesn't exist and update the tokens if it already exists, are we ok with that?

strager commented 1 year ago

So, merging this two functions will result in the removal of the check for a non existent user

The checks should still be present in get_access_token and get_refresh_token. update_or_create_user shouldn't ever throw UserNotFoundError.