-
-
Notifications
You must be signed in to change notification settings - Fork 116
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
Optional ETag and Content-Format in SetResponse #140
Comments
I also like the idea and it seems straight forward to implement. How should the API look like?Just give access to the response object of the response writer? |
You can already send a response without ETag and ContentType when using the Client() The only remaining side-effect is, that an ACK is send for Confirmable requests and you can not piggyback your response this way.
|
I'll need to dig back into this but I think the Confirmable request was actually the exact problem I was having where I needed to use w.SetResponse(...) instead of w.Client().WriteMessage(...) |
Related to: #349 |
Implemented in |
The ResponseWriter by default adds options for ETag and Content-Format if a body io.ReadSeeker is present.
We're very byte conscious and as neither are required by RFC 7252 it would be preferable if their inclusion was optional.
The text was updated successfully, but these errors were encountered: