miguelgrinberg / microblog-api

A modern (as of 2024) Flask API back end.
MIT License
365 stars 100 forks source link

Last seen not updated when DISABLE_AUTH=1 #26

Closed ghost closed 11 months ago

ghost commented 11 months ago

Hi Miguel, thanks for your contributions to the Flask community.

I was going through the code, and noticed that on this line, we ping the user, but not commit the change. This may be by design, I'm not sure, but since verify_access_token commits the last_seen, I just thought I'll point it out.

Thanks

miguelgrinberg commented 11 months ago

No particular reason. You could add a commit there if you wanted. Running with auth disabled is not something that is intended to ever be used, except when following the first few chapters of the React Mega-Tutorial.