Conversation
Owner
Author
|
related #43 |
Owner
Author
|
/lgtm |
1 similar comment
Owner
Author
|
/lgtm |
Owner
Author
|
/hold |
fix lint xml fail test case add Attachment func
Merging this branch will increase overall coverage
Coverage by fileChanged files (no unit tests)
Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code. Changed unit test files
|
Owner
Author
|
/lgtm |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds six top-level response helpers that write complete HTTP responses
(headers, status code, body) on top of
net/http:JSON(w, statusCode, data, opts...)— encodes withencoding/json;options:
EscapeForHTML(false)to disable HTML escaping (on by default)and
Indented(true)for pretty-printed outputXML(w, statusCode, data)— marshals withencoding/xml, writes theXML declaration; marshaling errors return before any header is sent
Text(w, statusCode, s)—text/plain; charset=utf-8viaio.WriteStringBytes(w, statusCode, contentType, b)— in-memory raw bytes with acaller-supplied content type
Stream(w, statusCode, contentType, r)—io.Copyfrom anyio.Readerwith constant memory; takes the sendfile fast path for
*os.File.Intended for large or not-yet-complete bodies (downloads, proxied
upstream responses, generated streams); disk files that need Range and
caching support should use
http.ServeFile/http.ServeFileFSAttachment(w, filename, r)— sends any io.Reader as a downloadattachment with 200 OK: Content-Disposition is derived from filename
(Unicode-safe via RFC 5987 through mime.FormatMediaType), Content-Type
is inferred from the extension with an application/octet-stream
fallback; built on Stream
Usage