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.

Sign up for GitHub

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

Stop useless panicking in context and render #2150

Merged
merged 16 commits into from
Feb 12, 2023
Merged

Stop useless panicking in context and render #2150

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
0709ae3
Removed useless panicking from renderer/context
eeonevision Nov 25, 2019
e0ca0b1
Added custom http status code to context's render
eeonevision Nov 25, 2019
edbd1ab
Updated context's unit tests
eeonevision Nov 25, 2019
f5c2b09
Fixed render test
eeonevision Nov 25, 2019
ea0b535
Removed broken pipe handling condition
eeonevision Nov 26, 2019
5381382
Returned public WriteJSON method
eeonevision Nov 27, 2019
f078613
Removed orphan http constant
eeonevision Nov 29, 2019
d36452f
Merge branch 'master' into master
appleboy Dec 1, 2019
4497ba5
Merge branch 'master' into master
eeonevision Dec 2, 2019
6a15de7
Merge branch 'master' into master
eeonevision Dec 9, 2019
7c501d5
Merge branch 'master' into master
eeonevision Jan 9, 2020
90ac8b4
Merge branch 'master' into master
eeonevision Jun 17, 2020
3dea0bc
Merge remote-tracking branch 'upstream/master'
eeonevision Jan 10, 2023
d0b7ad2
Fixed json renderer
eeonevision Jan 10, 2023
9a45526
Fixed error handling in context render
eeonevision Jan 10, 2023
f32da2d
Renamed panic render to render in context test
eeonevision Jan 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Returned public WriteJSON method
  • Loading branch information
@eeonevision
eeonevision committed Nov 27, 2019
commit 53813826951152f1433d72f3d2e972c7e2c55aa5
8 changes: 8 additions & 0 deletions render/json.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,14 @@ func (r JSON) WriteContentType(w http.ResponseWriter) {
writeContentType(w, jsonContentType)
}

// WriteJSON marshals the given interface object and writes it with custom ContentType.
func WriteJSON(w http.ResponseWriter, obj interface{}) error {
writeContentType(w, jsonContentType)
encoder := json.NewEncoder(w)
err := encoder.Encode(&obj)
return err
}

// Render (IndentedJSON) marshals the given interface object and writes it with custom ContentType.
func (r IndentedJSON) Render(w http.ResponseWriter) error {
r.WriteContentType(w)
Expand Down