Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

last_used/last_seen updated on all API requests #3231

Closed
Tracked by #121
, #122
...
davwheat opened this issue Dec 27, 2021 · 0 comments · Fixed by #3230
Closed
Tracked by #121
, #122
...

last_used/last_seen updated on all API requests #3231

davwheat opened this issue Dec 27, 2021 · 0 comments · Fixed by #3230
Assignees
@davwheat
Labels
type/performance
Milestone
1.2

Comments

@davwheat
Copy link
Member

davwheat commented Dec 27, 2021 •
edited

Access tokens last_used date and User last_seen at updated on each request, including for API GET requests, causing ALL requests to always use write DB connections and causing massive usage on databases
@davwheat davwheat changed the title access tokens accessed date and user last seen at are set on each request, including for api GET requests causing ALL requests to always use write db connections and causing massive usage on databases last_used/last_seen updated on all API requests Dec 27, 2021
@davwheat davwheat self-assigned this Dec 27, 2021
@davwheat davwheat mentioned this issue Dec 27, 2021
Merged
8 tasks
@davwheat davwheat added this to the 1.2 milestone Dec 27, 2021
This was referenced Mar 10, 2022
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davwheat davwheat

Labels
type/performance
Projects
None yet
Milestone
1.2
Development

Successfully merging a pull request may close this issue.

perf: only update last seen time when current > 120s ago flarum/framework
1 participant
@davwheat