Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make it possible to pass functions to change output streaming #116

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Make it possible to pass functions to change output streaming #116

wants to merge 12 commits into from

Commits on May 30, 2024

  1. Make it possible to pass functions to change output streaming

    @thekid
    thekid committed May 30, 2024
    Configuration menu
    Copy the full SHA
    e784512 View commit details
    Browse the repository at this point in the history

  2. QA: Shorten streaming test

    @thekid
    thekid committed May 30, 2024
    Configuration menu
    Copy the full SHA
    2e2846e View commit details
    Browse the repository at this point in the history

  3. Include streaming tests

    @thekid
    thekid committed May 30, 2024
    Configuration menu
    Copy the full SHA
    065f228 View commit details
    Browse the repository at this point in the history

Commits on Jun 2, 2024

  1. Merge branch 'master' into feature/streaming

    @thekid
    thekid committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    8dc1696 View commit details
    Browse the repository at this point in the history

  2. MFH fix for XDebug

    @thekid
    thekid committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    367f5d3 View commit details
    Browse the repository at this point in the history

Commits on Jun 9, 2024

  1. Also use streaming function when Response::flush() is called

    @thekid
    thekid committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    878674f View commit details
    Browse the repository at this point in the history

  2. QA: CS/WS consistency

    @thekid
    thekid committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    ca06077 View commit details
    Browse the repository at this point in the history

  3. Refactor Output::stream() to accept length and return one of the foll… 

    …owing:

. WriteLength, when length !== null
. WriteChunks, for HTTP/1.1
. Buffered, otherwise

This way, we do not need to set the Content-Length header inside Response and
can prevent code duplication in various places, including streaming functions,
at the cost of a tiny overhead when writing data with a known content length.
This overhead consists of additional method calls and instantiating and later
garbage-collecting an additional object, all of which is unmeasurable compared
to the network I/O
    @thekid
    thekid committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    2f0e426 View commit details
    Browse the repository at this point in the history

  4. Adjust stream() documentation, use $length for parameter name

    @thekid
    thekid committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    2041559 View commit details
    Browse the repository at this point in the history

  5. Type-hint implementation parameter for streaming()

    @thekid
    thekid committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    b1347cd View commit details
    Browse the repository at this point in the history

  6. Render exceptions in developer console

    @thekid
    thekid committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    4df76c4 View commit details
    Browse the repository at this point in the history

  7. Change from sprintf() for templating to a Mustache MVP

    @thekid
    thekid committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    ca7d4fd View commit details
    Browse the repository at this point in the history