[wp-trac] [WordPress Trac] #49291: Code issues with set_time_limit

[WordPress Trac]

#49291: Code issues with set_time_limit
-------------------------------------+-------------------------------------
 Reporter:  madpeter                 |       Owner:  (none)
     Type:  defect (bug)             |      Status:  reopened
 Priority:  normal                   |   Milestone:  6.8
Component:  Revisions                |     Version:  5.3.2
 Severity:  normal                   |  Resolution:
 Keywords:  needs-patch good-first-  |     Focuses:  administration,
  bug                                |  performance
-------------------------------------+-------------------------------------
Changes (by SergeyBiryukov):

 * keywords:   => needs-patch good-first-bug
 * milestone:  Awaiting Review => 6.8


Comment:

 This appears to be the only instance of `set_time_limit( 0 )` in core,
 i.e. removing the timeout limit completely.

 Other instances have a reasonable limit instead, e.g. `set_time_limit( 5 *
 MINUTE_IN_SECONDS )`.

 Looking at #24757 again, I think we could do the same here to avoid
 hanging processes.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/49291#comment:7>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform