LBRYCommunity/lbry.go
1
0
Fork 0
Code Issues 5 Pull requests Projects Releases 46 Packages Wiki Activity

Status error should default to internal server error. Otherwise it will trigger a nil pointer and subsequent panic.

Browse source
This commit is contained in:
Mark Beamer Jr 2021-04-21 13:58:38 -04:00
parent 2b155597bf
commit a1177c17d3
No known key found for this signature in database
GPG key ID: 1C314FB89AD76973
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
extras/api/server.go
View file

@ -121,6 +121,9 @@ func (h Handler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
if rsp.Error != nil { if rsp.Error != nil {
ogErr := errors.Unwrap(rsp.Error) ogErr := errors.Unwrap(rsp.Error)
if statusError, ok := ogErr.(StatusError); ok { if statusError, ok := ogErr.(StatusError); ok {
if statusError.Status == 0 {
statusError.Status = http.StatusInternalServerError
}
rsp.Status = statusError.Status rsp.Status = statusError.Status
} else { } else {
rsp.Status = http.StatusInternalServerError rsp.Status = http.StatusInternalServerError