Limits the number of logs returned. This closes #13.

loggingapi_test.go

@@ -35,6 +35,6 @@ func TestBasicCall(t *testing.T) {
 	}
 
 	if len(logs.GetLogs()) != 2 {
-		t.Errorf("bad number of logs: %v", logs)
+		t.Errorf("bad number of logs: (%v) %v", len(logs.GetLogs()), logs)
 	}
 }

loggingutils.go

@@ -5,6 +5,7 @@ import (
 	"io/ioutil"
 	"os"
 	"path/filepath"
+	"sort"
 	"strings"
 	"time"
 
@@ -55,7 +56,19 @@ func (s *Server) loadAllLogs(ctx context.Context, origin string) ([]*pb.Log, err
 		return nil
 	})
 
-	return logs, err
+	sort.SliceStable(logs, func(i, j int) bool {
+		return logs[i].GetTimestamp() < logs[j].GetTimestamp()
+	})
+
+	// Only return 20 logs
+	return logs[0:min(20, len(logs))], err
+}
+
+func min(a, b int) int {
+	if a < b {
+		return a
+	}
+	return b
 }
 
 func (s *Server) loadLogs(ctx context.Context, origin string, timestamp int64) ([]*pb.Log, error) {

loggingutils_test.go

@@ -40,3 +40,9 @@ func TestBadFullLoad(t *testing.T) {
 	}
 
 }
+
+func TestMin(t *testing.T) {
+	if min(1, 10) != 1 || min(10, 1) != 1 {
+		t.Errorf("Min is wrong")
+	}
+}