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

Size of response with no 'content-length' header #102

Open
Nokel81 opened this issue Apr 7, 2016 · 7 comments · May be fixed by #241
Open

Size of response with no 'content-length' header #102

Nokel81 opened this issue Apr 7, 2016 · 7 comments · May be fixed by #241

Comments

@Nokel81
Copy link

Nokel81 commented Apr 7, 2016

Is it possible for a tag to be added that uses 'content-length' is the type of the data sent is not in chunks and a chunk size indicator when the data is sent in chunks?

@dougwilson
Copy link
Contributor

It probably could. Please create a pull request and we'd be happy to take a look & accept!

@mum-never-proud
Copy link

I would like to raise a pr if no ones working on this

any rough idea on how the tag to be named?

@Nokel81
Copy link
Author

Nokel81 commented Mar 29, 2020

computed-size?

It could even default to size if that was provided.

@mum-never-proud
Copy link

mum-never-proud commented Mar 29, 2020

hmm, but how computed-size can help differentiate between a response is in chunks or not?

may I didn't get your point on this could you please explain

::1 - - [28/Mar/2020:16:19:54 +0000] "GET / HTTP/1.1" 200 - "-" "insomnia/7.1.1"

having this as the response how can we differentiate the response is chunked or not

thanks!

can we make use of Transferred-Encoding header here to highlight the type of transfer?

@Nokel81
Copy link
Author

Nokel81 commented Mar 30, 2020

Probably, I was more thinking of how to get the information of the computed size to the user

@mum-never-proud
Copy link

@dougwilson any pointers on this?

@dougwilson
Copy link
Contributor

Just raise the PR; if there is some issue with the choosen token name it can be addressed in review 👍

@mum-never-proud mum-never-proud linked a pull request Apr 18, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants