Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#23811 closed defect (bug) (fixed)

XML-RPC shouldn't display errors

Reported by: koke's profile koke Owned by: markjaquith's profile markjaquith
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.5.1
Component: XML-RPC Keywords: mobile has-patch
Focuses: Cc:


I've got some user reports of XML-RPC returning this:

<a href='function.strpos'>function.strpos</a>

In this case, it seems to be #18310, but XML-RPC shouldn't be throwing any warnings, since that makes the client think the request failed when it actually succeeded

Attachments (4)

23811.diff (453 bytes) - added by markoheijnen 11 years ago.
Possible fix
23811.2.diff (719 bytes) - added by kovshenin 11 years ago.
23811.3.diff (1006 bytes) - added by nacin 11 years ago.
23811.4.diff (479 bytes) - added by markjaquith 11 years ago.
once, after

Download all attachments as: .zip

Change History (11)

11 years ago

Possible fix

#1 @markoheijnen
11 years ago

  • Milestone changed from Awaiting Review to 3.6

Moving to 3.6 for future discussions.

11 years ago

#3 @kovshenin
11 years ago

  • Keywords has-patch added

23811.2.diff is similar to Marko's patch but for the WP_DEBUG_DISPLAY constant instead, which then sets display_errors.

#4 @markoheijnen
11 years ago

That doesn't exactly works the same. Your patch only works when WP_DEBUG is true and my one only works when WP_DEBUG is false.

I guess the issue here is that the server definition is display_errors: 1. So I guess both patches make sense to commit then?

#5 @SergeyBiryukov
11 years ago

23811.diff makes sense to me.

Not sure if WP_DEBUG_DISPLAY should default to false for XML-RPC. Should we do the same for AJAX in that case? I guess it should still default to true, otherwise it would make debugging harder.

Last edited 11 years ago by SergeyBiryukov (previous) (diff)

11 years ago

11 years ago

once, after

#6 @markjaquith
11 years ago

  • Owner set to markjaquith
  • Resolution set to fixed
  • Status changed from new to closed

In 24564:

Don't display errors for XML-RPC requests, as they malform the response.

fixes #23811. props kovshenin, markoheijnen.

#7 @nacin
11 years ago

In 24581:

Documentation for [24564]. see #23811.

Note: See TracTickets for help on using tickets.