Skip to content

Commit b7ea8dc

Browse files
committed
Handled Errors in server
1 parent bdcad4b commit b7ea8dc

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

cmd/api.go

+6-2
Original file line numberDiff line numberDiff line change
@@ -54,19 +54,23 @@ func Handler(w http.ResponseWriter, r *http.Request) {
5454
result = site.Search(search)
5555
} else if list != "" {
5656
log.Debug("listing page ", list)
57-
pagenum, _ := strconv.Atoi(list)
57+
pagenum, err := strconv.Atoi(list)
58+
if err != nil {
59+
http.Error(w, "Page must be a number", http.StatusBadRequest)
60+
}
5861
result = site.List(pagenum)
5962
}
6063

6164
// dump results
6265
b, err := json.Marshal(result.Movies)
6366
if err != nil {
6467
log.Fatal("failed to serialize response: ", err)
68+
http.Error(w, "Internal Server Error", http.StatusInternalServerError)
6569
}
6670
enableCors(&w)
6771
w.Header().Add("Content-Type", "application/json")
6872
w.Write(b)
69-
log.Debug("Completed search for", search, list)
73+
log.Debug("Completed search for ", search, list)
7074
}
7175

7276
func enableCors(w *http.ResponseWriter) {

0 commit comments

Comments
 (0)