Fix encoding stream size

[ajgarlag]

Q	A
Bug fix?	yes
New feature?	no
BC breaks?	no
Deprecations?	no
Related tickets	fixes #60
License	MIT

What's in this PR?

This PR includes the tests required to expose the problem described in #60 about the invalid getSize returned value by FilteredStream subclasses, and should include a fix.

Why?

#60 Invalid value returned by FilteredStream::getSize

Checklist

• Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix

To Do

• Discuss and find a solution to fix the tests
• Squash commits

[ajgarlag]

A quick fix could be to implement a getSize method in FilteredStream to always return null (allowed by PSR-7 when the size is unknown) and rewrite the tests to expect that null value.

[joelwurtz]

AFAIK, result size on the compression cannot be determined, but they may be headers for the decompression (not sure).

So 👍 for the null value as i don't see a way to do this unless we read up the whole stream ^^ which will make it non readable as we cannot seek a filtered stream (don't remember where is see this but using \php_user_filter make a stream not seekable)

[dbu]

that looks at least not wrong to me. i am 👍 to merge this. for future reference, can we do better than this for some of the stream types?

[joelwurtz]

LGTM,

Could you rebase with the last PR merged (to remove conflict in the changelog), then it's fine for me

[sagikazarmark]

Sorry for the late response. Thanks.