Bug fix and TODO

internal/web/web.go

@@ -87,10 +87,15 @@ func (web *Web) AnalysisHandler(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
+	// TODO there may be a scenario where a diff isn't return (after a forced
+	// push?), if so, we should just give the template the issues to render.
+	// If no errors, give template nil issues.
+
 	diffReader, err := vcs.Diff(r.Context(), analysis.RepositoryID, analysis.CommitFrom, analysis.CommitTo, analysis.RequestNumber)
 	if err != nil {
 		log.Printf("error getting diff from VCS for analysisID %v: %v", analysisID, err)
 		web.errorHandler(w, r, http.StatusInternalServerError, "Could not get VCS")
+		return
 	}
 	defer diffReader.Close()