Make WordPress Core

Opened 19 years ago

Closed 19 years ago

#1166 closed defect (bug) (fixed)

Pingbacks don't work with allow_url_fopen=Off

Reported by: anonymousbugger's profile anonymousbugger Owned by: matt's profile matt
Milestone: Priority: normal
Severity: major Version: 1.5
Component: XML-RPC Keywords:
Focuses: Cc:

Description

pingback_ping() uses @fopen($pagelinkedfrom, 'r') to check if the source URI exists. This check always fails if allow_url_fopen is Off (recommended setting), that means pingbacks don't work.

Please consider replacing this fopen call with something like the code from http://7sheaven.pingouin.ca/forum/showthread.php?t=13

Attachments (1)

foo.php (162 bytes) - added by anonymousbugger 19 years ago.

Download all attachments as: .zip

Change History (11)

#2 @anonymousbugger
19 years ago

  • Patch set to No

#3 @matt
19 years ago

  • Owner changed from anonymous to matt
  • Resolution changed from 10 to 50
  • Status changed from new to closed

I can't use the code from the link if the link doesn't work.

#4 @anonymousbugger
19 years ago

  • Resolution changed from 50 to 30
  • Status changed from closed to assigned

Just use snoopy instead of fopen($url)

#5 @MC_incubus
19 years ago

Agreed. We're bundling Snoopy with WordPress, and using it will increase compat.

#6 @matt
19 years ago

So remote fwrites work even if remote fopen is off?

#7 @matt
19 years ago

Dreamhost, which recently disabled remote fopen, is suggesting users use curl. Curl is only used in Snoopy for SSL connections.

We may also want to upgrade snoopy.

#8 @anonymousbugger
19 years ago

I think the CURL PHP module is the way to go.

The module is available on a lot of hosts, the 'curl' binary isn't (especially in chroot environments). So if you continue to use Snoopy, it should be modified to use the module instead of the binary.

#9 @matt
19 years ago

Possible fix checked in, please confirm

#10 @matt
19 years ago

  • fixed_in_version set to 1.5.1
  • Resolution changed from 30 to 20
  • Status changed from assigned to closed

Resolving pending further feedback

@anonymousbugger
19 years ago

Note: See TracTickets for help on using tickets.