guzzle / guzzle

Guzzle, an extensible PHP HTTP client

Home Page:https://docs.guzzlephp.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Never cache thru disk implicitly

mvorisek opened this issue · comments

Description

In a few places in this lib and guzzle/psr7 repo php://temp stream is used to store temporary data.

Caching thru disk by default is not expected by a HTTP library.

This is a feature request to change all php://temp to php://memory.

If user needs to process huge files, he should pass php://temp manually with awareness it can be slow and imply huge disk IO.

(related with #3115 as stream processing is the best option vs. buffering the whole response)

Caching thru disk by default is not expected by a HTTP library.

[citation needed]

It is my opinion, at least I do not want any disk IO for temporary data when HTTP download is involved.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 2 weeks if no further activity occurs. Thank you for your contributions.