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.

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

Allow stream :content #19

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

Commits on Nov 1, 2015

  1. Allow stream as :content

    Bear in mind that there will be an issue if the contents is read and then,
    for whatever reason, the auto-restart is triggered.
    wemeetagain committed Nov 1, 2015
    Configuration menu
    Copy the full SHA
    483b6a5 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    730ee4a View commit details
    Browse the repository at this point in the history
  3. allow :content-length to be set

    If :content-length is set, and :content is a stream, only copy
    that many bytes, otherwise, set transfer-encoding to chunked, and copy
    the entirety of the stream.
    wemeetagain committed Nov 1, 2015
    Configuration menu
    Copy the full SHA
    4578f36 View commit details
    Browse the repository at this point in the history

Commits on Nov 2, 2015

  1. fix header-writing logic

    If :content is a stream and :content-length is a specified header,
    set :content-length and copy that many bytes.
    
    If :content is a stream and :content-length is not specified,
    set :transfer-encoding to chunked, and copy the entire stream.
    
    The :content-type in both cases is defaulted to "application/octet-stream"
    wemeetagain committed Nov 2, 2015
    Configuration menu
    Copy the full SHA
    4a4b4c8 View commit details
    Browse the repository at this point in the history