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

Bug 12 #13

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

Bug 12 #13

wants to merge 3 commits into from

Conversation

nponeccop
Copy link

The PR contains 3 changes:

  • travis.yml so bugs like this are prevented in the future
  • The transfer encoding patch from @rkaw92 which improves caching support in many cases
  • Closes mime error #12

rkaw92 and others added 3 commits April 28, 2020 18:56
…d files

This change allows clients (such as XHR requests) to obtain the total size of the served file, e.g. to display a progress bar.
One caveat that applies is that the HTTP spec does not define if Content-Length refers to the compressed or uncompressed content.
Here, we arbitrarily pick the compressed content (body length verbatim), as it is the most readily-available measure for the client.
You should still enable github integration in Travis UI
Mime 2.x deleted charset support, as it isn't very useful now
(almost everyone has moved to UTF-8). So
we can either delete the charset header or use a different library
which does provide the charset API.

Closes andrewrk#12
@antonio-castellon
Copy link

Could someone agree on this commit and release it on master ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mime error
3 participants