Milo: Fix ?limit= param

milo/frontend/middleware.go

 // Copyright 2017 The LUCI Authors. All rights reserved.
 // Use of this source code is governed under the Apache License, Version 2.0
 // that can be found in the LICENSE file.
 
 package frontend
 
 import (
         "bytes"
         "fmt"
         "html/template"
@@ skipping 234 matching lines @@
                 return -1, fmt.Errorf("limit parameter value %q is less than 0", sLimit)
         }
         return limit, nil
 }
 
 // pagedURL returns a self URL with the given cursor and limit paging options.
 // if limit is set to 0, then inherit whatever limit is set in request.  If
 // both are unspecified, then limit is omitted.
 func pagedURL(r *http.Request, limit int, cursor string) string {
         if limit == 0 {
-                var err error
-                limit, err = GetLimit(r)
-                if err != nil {
-                        // This should not happen because the handler should've already validated the
-                        // limit earlier in the process.
-                        panic(err)
-                }
+                limit, _ = GetLimit(r) // This returns -1 on error.
                 if limit < 0 {
                         limit = 0
                 }
         }
         values := r.URL.Query()
         switch cursor {
         case "EMPTY":
                 values.Del("cursor")
         case "":
                 // Do nothing, just leave the cursor in.
@@ skipping 100 matching lines @@
         c.Context = withRequest(c.Context, c.Request)
         next(c)
 }
 
 func getRequest(c context.Context) *http.Request {
         if req, ok := c.Value(&requestKey).(*http.Request); ok {
                 return req
         }
         panic("No http.request found in context")
 }