whythawk / full-stack-fastapi-postgresql

Full stack, modern web application generator. Using FastAPI, PostgreSQL as database, Nuxt3, Docker, automatic HTTPS and more.
MIT License
231 stars 47 forks source link

undefined function in /validate-email:post #17

Closed FranzForstmayr closed 1 year ago

FranzForstmayr commented 1 year ago

The function verify_password_reset_token is not defined, but used in validate_email endpoint.

Is this endpoint not needed anymore through the new auth scheme?

turukawa commented 1 year ago

Yeah, it's an artifact left over from some not-quite-completed thought. Something about an admin creating a user-account, and then the user needing to validate their email after login, but that workflow seems a bit dodgy with the new auth scheme.

There's a few functions supporting that process needing removal. I'll track them down and excise them.

This is where having a rebuilt test suite would be great ;)