feat: Allow to get Probe logs by target

main.go

@@ -212,14 +212,31 @@ func run() int {
 	http.HandleFunc(path.Join(*routePrefix, "/logs"), func(w http.ResponseWriter, r *http.Request) {
 		id, err := strconv.ParseInt(r.URL.Query().Get("id"), 10, 64)
 		if err != nil {
-			http.Error(w, "Invalid probe id", 500)
-			return
+			id = -1
 		}
-		result := rh.Get(id)
-		if result == nil {
-			http.Error(w, "Probe id not found", 404)
+		target := r.URL.Query().Get("target")
+		result := new(prober.Result)
+		if err == nil && target != "" {
+			http.Error(w, "Probe id and target can't be defined at the same time", http.StatusInternalServerError)
+			return
+		} else if id == -1 && target == "" {
+			http.Error(w, "Probe id or target must be defined as http query parameters", http.StatusInternalServerError)
 			return
 		}
+		if target != "" {
+			result = rh.GetByTarget(target)
+			if result == nil {
+				http.Error(w, "Probe target not found", http.StatusNotFound)
+				return
+			}
+		} else {
+			result = rh.GetById(id)
+			if result == nil {
+				http.Error(w, "Probe id not found", http.StatusNotFound)
+				return
+			}
+		}
+
 		w.Header().Set("Content-Type", "text/plain")
 		w.Write([]byte(result.DebugOutput))
 	})
@@ -230,7 +247,7 @@ func run() int {
 		sc.RUnlock()
 		if err != nil {
 			level.Warn(logger).Log("msg", "Error marshalling configuration", "err", err)
-			http.Error(w, err.Error(), 500)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
 		w.Header().Set("Content-Type", "text/plain")

prober/history.go

@@ -78,8 +78,8 @@ func (rh *ResultHistory) List() []*Result {
 	return append(rh.preservedFailedResults[:], rh.results...)
 }
 
-// Get returns a given result.
-func (rh *ResultHistory) Get(id int64) *Result {
+// Get returns a given result by id.
+func (rh *ResultHistory) GetById(id int64) *Result {
 	rh.mu.Lock()
 	defer rh.mu.Unlock()
 
@@ -96,3 +96,22 @@ func (rh *ResultHistory) Get(id int64) *Result {
 
 	return nil
 }
+
+// Get returns a given result by url.
+func (rh *ResultHistory) GetByTarget(target string) *Result {
+	rh.mu.Lock()
+	defer rh.mu.Unlock()
+
+	for _, r := range rh.preservedFailedResults {
+		if r.Target == target {
+			return r
+		}
+	}
+	for _, r := range rh.results {
+		if r.Target == target {
+			return r
+		}
+	}
+
+	return nil
+}