AllenNeuralDynamics / AllenNeuralDynamics/aind-data-transfer-service

Add session `max_age` to SessionMiddleware

Open
#267 1 comment 0 reactions 1 assignee Claimed by @yosefmaru View on GitHub
Dominant language
Python
Stars
3
Forks
0
Avg merge
11m
Merged PRs (30d)
1

Description

**Is your feature request related to a problem? Please describe.**
Currently, when the user logs in, their `userinfo` (not full token) is saved in the request session. This is saved until the user logs out, or the browser session is cleared, or the default session `max_age` of 2 weeks.

**Describe the solution you'd like**
The request session should have a shorter `max_age`. Consider having it match the auth token timeout?

**Describe alternatives you've considered**
Save and refresh the auth token if it has expired.

**Additional context**
See https://www.starlette.io/middleware/#sessionmiddleware:
> max_age - Session expiry time in seconds. Defaults to 2 weeks. If set to None then the cookie will last as long as the browser session.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.