Not a bug Missing notifications because execution time limit is hard coded to 3 seconds

[W1zzard]

Affected version

2.1.2

XF\Service\AbstractNotifier::notify takes a parameter "$timeLimit", which seems to be hardcoded to 3 seconds by all callers.

For threads with many notifications this causes it to time out and people keep wondering why they don't get their notifications.

Might want to run this in a background task, or least give us a config option to change the timeout, please.

 

[Chris D]

Might want to run this in a background task, or least give us a config option to change the timeout, please.

That's what the time limit is for. If the time limit is exceeded, a background job is created to finish the work.

 

[W1zzard]

Oh, my bad then