Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#27668 closed defect (bug) (fixed)

When using HTTP Streams, check if parse_url returned a path.

Reported by:	DvanKooten	Owned by:	dd32
Priority:	normal	Milestone:	4.1
Component:	HTTP API	Version:	3.8.1
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:

Description

When using HTTP Streams to do a remote request to a domain (with no path) PHP's parse_url doesn't return an array with a path key.

When building the $requestPath variable, we should check if the path is set before concatenating strings.

Attachments (1)

27668.diff (860 bytes ) - added by barrykooij 12 years ago.

Download all attachments as: .zip

Change History (5)

@barrykooij
12 years ago

Attachment 27668.diff added

#1 @barrykooij
12 years ago

Keywords has-patch added

#2 @barrykooij
12 years ago

Keywords needs-testing added

#3 @dd32
12 years ago

Owner set to dd32
Resolution → fixed
Status new → closed

Avoid a PHP Notice when requesting a url without a path component (http://example.com) in the Streams HTTP component.
Props barrykooij for inital patch; Fixes #27668

#4 @dd32
12 years ago

Keywords needs-testing removed
Milestone Awaiting Review → 4.1

Note: See TracTickets for help on using tickets.

Download in other formats: