View Issue Details

IDProjectCategoryView StatusLast Update
0005665Composr non-bundled addons[All Projects] General / Uncategorisedpublic2024-09-13 03:23
ReporterPatrick SchmalstigAssigned ToPatrick Schmalstig 
SeverityMinor-bug 
Status resolvedResolutionfixed 
Summary0005665: Hybridauth: maximum execution time on session_start
DescriptionSometimes PHP times out (10 seconds) when calling session_start . IMO it is unreasonable to extend the time limit for this. We should analyze why it is taking so long.
TagsRoadmap: v11
Time estimation (hours)
Sponsorship open

Activities

Chris Graham

2024-07-25 22:53

administrator   ~0008966

This is a locking problem. Something else (or the same thing in parallel) is calling session_start and not closing the session quickly.

Patrick Schmalstig

2024-07-25 23:25

administrator   ~0008971

It was becoming a big issue for me. Temporarily, I changed the code so that a Hybridauth session doesn't initialize except when Hybridauth itself is initialized... instead of doing it in the startup hook on each and every page load.

That seemed to do the trick at least so I can get through testing without a bunch of timeouts, but I'm assuming we can't leave it like that? I looked everywhere on Hybridauth's docs and found nothing mentioning use of sessions, so do we really need that?

Chris Graham

2024-07-26 01:15

administrator   ~0008973

Hybridauth uses sessions to track oAuth/etc key exchanges through hops. So, yes, it's needed, unless we replaced with a DB backend.
Your fix sounds correct.

Patrick Schmalstig

2024-09-13 03:23

administrator   ~0009369

Marking as resolved... the temporary fix is working well. Will keep as permanent.

Issue History

Date Modified Username Field Change
2024-04-03 23:57 Patrick Schmalstig New Issue
2024-04-03 23:57 Patrick Schmalstig Status non-assigned => assigned
2024-04-03 23:57 Patrick Schmalstig Assigned To => Patrick Schmalstig
2024-07-25 22:53 Chris Graham Note Added: 0008966
2024-07-25 22:54 Chris Graham Tag Attached: Roadmap: v11
2024-07-25 23:25 Patrick Schmalstig Note Added: 0008971
2024-07-26 01:15 Chris Graham Note Added: 0008973
2024-09-13 03:23 Patrick Schmalstig Status assigned => resolved
2024-09-13 03:23 Patrick Schmalstig Resolution open => fixed
2024-09-13 03:23 Patrick Schmalstig Note Added: 0009369