Make WordPress Core

Opened 4 years ago

Last modified 14 months ago

#17619 new enhancement

Soft 404 at /wp-content/plugins/

Reported by: miqrogroove Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch dev-feedback
Focuses: Cc:


/wp-content/plugins/index.php would be better written with a proper 404.

header('HTTP/1.0 404 Not Found'); // Silence is golden.

Attachments (3)

17619.diff (341 bytes) - added by BenChapman 4 years ago.
Adds a 404 error message
17619.patch (664 bytes) - added by jakub.tyrcha 4 years ago.
17619.2.patch (1005 bytes) - added by jakub.tyrcha 4 years ago.

Download all attachments as: .zip

Change History (14)

#1 @johnbillion
4 years ago

Shouldn't it technically be a 403?

#2 @miqrogroove
4 years ago

Technically, there is nothing in the response entity, so either 403 or 404 is appropriate.

#3 @toscho
4 years ago

  • Cc info@… added

Why not 410? Nobody should come back, no?

#4 @BenChapman
4 years ago

As per RFC2616 if a reason is to be given for denying access then 403 should be used, otherwise 404 should be used. In this case I would say 403 would be the better response code with a reason included for forbidding access.

Version 0, edited 4 years ago by BenChapman (next)

4 years ago

Adds a 404 error message

#5 @BenChapman
4 years ago

  • Cc admin@… added

#6 @solarissmoke
4 years ago

If this is going to change, then it should change for all wp-content directories, i.e., /themes and /uploads also.

#7 @DH-Shredder
4 years ago

  • Type changed from defect (bug) to enhancement

4 years ago

#8 @jakub.tyrcha
4 years ago

Why should we include this in the uploads/ directory as well?

#9 @edwardw
4 years ago

  • Keywords has-patch added

I feel that a 403 would be a more appropriate response since there is actually a directory, just not that one permit access to access it.

Last edited 4 years ago by edwardw (previous) (diff)

#10 @chriscct7
14 months ago

  • Keywords dev-feedback added
  • Severity changed from minor to normal

#11 @DrewAPicture
14 months ago

@nacin: Possible to get a yea or nay here?

Note: See TracTickets for help on using tickets.