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

Expose chunked_threshold on Response. Fix #149 #150

Merged
merged 1 commit into from Dec 21, 2018

Conversation

lolgesten
Copy link

No description provided.

@lolgesten
Copy link
Author

@tomaka @frewsxcv I apologise for pinging you, I know you're probably both very busy.

Does anyone have any opinion on this issue? Does it go against the design goals? Can it be merged?

tomaka
tomaka previously requested changes Nov 1, 2018
src/response.rs Outdated Show resolved Hide resolved
src/response.rs Outdated Show resolved Hide resolved
@lolgesten
Copy link
Author

Hi @tomaka did my update address your concerns?

@lolgesten
Copy link
Author

@tomaka Sorry to chase. Is there anything needed to land this?

@frewsxcv frewsxcv dismissed tomaka’s stale review December 21, 2018 02:50

comments have been addressed

@frewsxcv frewsxcv merged commit 6980147 into tiny-http:master Dec 21, 2018
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.

None yet

4 participants