pallets-eco / flask-session

Server side session extension for Flask
https://flask-session.readthedocs.io
BSD 3-Clause "New" or "Revised" License
501 stars 239 forks source link

check expiry is not none #195

Closed PizzaPiee closed 8 months ago

PizzaPiee commented 11 months ago

Hi! I found this issue while working with the library

    if saved_session and saved_session.expiry <= datetime.utcnow():
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: '<=' not supported between instances of 'NoneType' and 'datetime.datetime'
armurox commented 9 months ago

Hi, sorry to bother, but could we get a follow up on this? I have encountered the same bug, and it's preventing me from deploying my app. I've fixed the problem in my local folder, but it doesn't help in vercel deployment. Does anyone have a recommendation for this?

Lxstr commented 8 months ago

Hi @PizzaPiee @armurox please test latest version and confirm if your issues are fixed. Thanks

armurox commented 8 months ago

Hi @PizzaPiee @armurox please test latest version and confirm if your issues are fixed. Thanks

Hi @Lxstr, yes, can confirm that my issue is fixed, session management is working with my database.