WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#31740 closed enhancement (wontfix)

Add multi_curl method to wp_remote_get

Reported by: ahansson89 Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: HTTP API Keywords:
Focuses: performance Cc:

Description

It would be nice to have a way to utilize multi_curl in a wp_remote_get request as it will speed up the execution time significantly when relying on multiple remote requests.

Change History (4)

#1 @dd32
6 years ago

  • Component changed from General to HTTP API
  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Due to the complexity of implementing it for both cURL and Streams, I'm marking this as wontfix, instead of adding extra complexity to the library.

It's a potential that we might move to another well maintained HTTP library at some point, and multi-get would be a nice-to-have feature then.

#2 @ahansson89
6 years ago

So this can happen once we move on from php 5.2 right?

#3 @rmccue
6 years ago

Actually, it's possible in 5.2: https://github.com/rmccue/Requests (one such library that @dd32 alludes to)

However, it's a significant amount of work to get it playing nicely across everything, and the HTTP API is a large piece of code with a small number of eyes watching it right now. Someday in the future though, it's absolutely a possibility (5.2 or not), just needs a chunk of time allocated to it. :)

#4 @ahansson89
6 years ago

Yea I was looking at that one too actually.

The HTTP API is for sure a large piece of code, and I understand why there is not many resources being allocated to it right now.

Note: See TracTickets for help on using tickets.