Use raw status codes when calling ctx.Handle

[thehowl]

Almost every instance of Context.Handle uses a raw HTTP status code - either 404 or 500. For consistency, I replaced the few instances where http.Status* was still used.

[codecov-io]

Codecov Report

Merging #3330 into master will decrease coverage by <.01%.
The diff coverage is 5.26%.

@@            Coverage Diff             @@
##           master    #3330      +/-   ##
==========================================
- Coverage   35.04%   35.04%   -0.01%     
==========================================
  Files         280      280              
  Lines       40556    40556              
==========================================
- Hits        14213    14211       -2     
- Misses      24237    24239       +2     
  Partials     2106     2106

Impacted Files	Coverage Δ
routers/repo/issue_watch.go	0% <0%> (ø)	⬆️
routers/repo/http.go	38.01% <0%> (ø)	⬆️
routers/repo/issue_timetrack.go	0% <0%> (ø)	⬆️
routers/repo/issue_stopwatch.go	26.66% <25%> (ø)	⬆️
models/repo_list.go	65.62% <0%> (-1.57%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f2b841d...9e6ff2f. Read the comment docs.

[strk]

I would actually prefer the other way around. Descriptive names are better than magic numbers IMHO

[thehowl]

However, I would argue that Go is not java, and I think it's pretty tedious/verbose to write http.StatusInternalServerError and http.StatusNotFound all over the place.

[bkcsoft]

http.Status* is the de facto Go standard. Use them

[lafriks]

I would better see ctx.NotFound and ctx.InternalServerError added

[thehowl]

I tend to agree with @lafriks. I'll make a PR removing ctx.Handle and replacing all instances with NotFound and InternalServerError. Closing this.