Skip to content
This repository has been archived by the owner on Feb 22, 2024. It is now read-only.

Add configuration for token expiration #322

Merged
merged 1 commit into from
May 2, 2015
Merged

Add configuration for token expiration #322

merged 1 commit into from
May 2, 2015

Conversation

waltaskew
Copy link
Contributor

Add the ability to set token expiration by configuring the SECURITY_TOKEN_MAX_AGE value.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 897b2fc on waltaskew:develop into 76ad77a on mattupstate:develop.

@waltaskew
Copy link
Contributor Author

Any chance on this getting in, @mattupstate ?

@mafrosis
Copy link
Contributor

mafrosis commented Dec 2, 2014

+1 I also need tokens to expire

@doender
Copy link

doender commented Apr 29, 2015

+1

mattupstate pushed a commit that referenced this pull request May 2, 2015
Add configuration for token expiration
@mattupstate mattupstate merged commit e4d9d3a into pallets-eco:develop May 2, 2015
@mattupstate
Copy link
Collaborator

Thanks for this.

@pdonorio
Copy link

I was trying to use the "SECURITY_TOKEN_MAX_AGE" found here with the pull of latest code.

Using "1 min":
The token is new at login after one minute, but the old tokens still work.
Am i doing something wrong?

@jonafato
Copy link
Collaborator

@pdonorio per the docs included, this should be number of seconds (an integer, not a human-readable string).

@pdonorio
Copy link

this should be number of seconds (an integer, not a human-readable string)

Tested, and you are right. Thanks @jonafato !

@galvarado
Copy link

Hi, this feature is not available on the current Version (1.7.4), right? When will be another release? The newest was on October 13th 2014. Many things have been improved since that day, as the flask-login backward compatibility and the token expiration, but as far as I see all the commits and PR's are on develop branch.Thanks for your time @mattupstate . (+1 For this excellent plugin)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants