WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#16959 closed defect (bug) (maybelater)

Core HTTP Transports implementation can support $url in test().

Reported by: hakre Owned by:
Priority: normal Milestone:
Component: HTTP API Version: 3.1
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

A sort of Follow-Up: With [17550], those functions are called with the addiotnal $url parameter. I think the change is useful.

Attachments (1)

16959.patch​ (1.5 KB ) - added by hakre 16 years ago.
adding $url

Download all attachments as: .zip

Change History (5)

@hakre
16 years ago

adding $url

#1 @hakre
16 years ago

Related: #11613

#2 @dd32
16 years ago

  • Milestone Awaiting Review
  • Resolution → maybelater
  • Status new → closed

When it's needed, or is useful, it's available.

#3 @dd32
16 years ago

Backstory: It was originally added for the purpose of proxy-blocking in the streams or fopen transport, but since we now require PHP5.2.4, it was no longer needed (And I forgot to remove it from the called location)

Since it neither adds, or removes functionality/need, theres no need for a change here

#4 @hakre
16 years ago

My Backstory: I didn't know it's passed to there as it was hidden in code. But it's good to know as test() implementations still can actually benefit from it.

Actually I was looking for it earlier today in #16606, passing of the URL allows to check for the actual stream transport. So I've added another patch: 16606.3.patch​

Note: See TracTickets for help on using tickets.