Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#32476 closed defect (bug) (invalid)

fix to tempnam() function breaks maintenance mode?

Reported by: cloudware's profile cloudware Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.2.2
Component: Upgrade/Install Keywords:
Focuses: Cc:

Description

This is a follow-up to #32135. Ref. Comment 4

The fix to the above bug appears to use the time() function to generate a temporary name rather than keep the name "/.maintenance"...

This surely will prevent WordPress from actually reporting to the users that the site is under maintenance, as it will be looking for a .maintenance file?

In some situations (DB upgrades etc) on busy sites the user interaction could have the potential for database corruption if the user is not prevented from access with the 'Under maintenance' page.

Keith

Change History (1)

#1 @dd32
11 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

wp_tempnam() isn't designed to create a specific file, it's used for creating a random temporary file, the filepath parameter you can pass is designed to be used as a namespace/seed for that created file.

Calling wp_tempnam("/.maintenance") doesn't achieve the intended effect, and WordPress itself never makes such a call. The .maintenance file is always created by concatenating ABSPATH and .maintenance.

Note: See TracTickets for help on using tickets.