search

mkhorasani / Streamlit-Authenticator

A secure authentication module to validate user credentials in a Streamlit application.
Apache License 2.0
1.38k stars 229 forks source link

Consider changing cookie_expiry_days to cookie_expiry_secs? #67

Closed voidel closed 1 year ago

voidel commented 1 year ago

So that we may define smaller time ranges for expiry, i.e. 5 minutes.

Timestamp in the JWT supports seconds granularity but the current param defines expiry in days.

Furthermore the type of the days expiry is integer, which prevents us from achieving the same goal with a decimal.

mkhorasani commented 1 year ago

Hi @voidel, the function annotation requires the expiry days parameter to be an integer, but that is non-binding and you can in fact input the expiry days as a float. Thanks.