Milo: Fix more builds link in builders page

milo/appengine/buildbucket/html.go

 // Copyright 2016 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 buildbucket
 
 import (
         "net/http"
 
         "github.com/julienschmidt/httprouter"
         "golang.org/x/net/context"
 
         "github.com/luci/luci-go/milo/appengine/settings"
         "github.com/luci/luci-go/milo/common/miloerror"
         "github.com/luci/luci-go/server/templates"
 )
 
 // TODO(nodir): move this value to luci-config.
 const defaultServer = "cr-buildbucket.appspot.com"
 
 // Builder displays builder view by fetching builds from buildbucket.
 type Builder struct{}
 
 // GetTemplateName for Builder returns the template name for builder pages.
 func (b Builder) GetTemplateName(t settings.Theme) string {
         return "builder.html"
 }
 
 // Render renders builder view page.
+// Note: The builder html template contains self links to "?limit=123", which could
+// potentially override any other request parameters set.
 func (b Builder) Render(c context.Context, r *http.Request, p httprouter.Params) (*templates.Args, error) {
         // Parse URL parameters.
         server := r.FormValue("server")
         if server == "" {
                 server = defaultServer
         }
 
         bucket := p.ByName("bucket")
         if bucket == "" {
                 return nil, &miloerror.Error{
@@ skipping 21 matching lines @@
         if err != nil {
                 return nil, err
         }
 
         // Render into the template
         args := &templates.Args{
                 "Builder": result,
         }
         return args, nil
 }