Buildbot build json grpc endpoint

milo/appengine/buildbot/html.go

 // Copyright 2015 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 buildbot
 
 import (
         "fmt"
         "net/http"
         "os"
+        "strconv"
 
         "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"
 )
 
 // Build is the container struct for methods related to buildbot build pages.
@@ skipping 23 matching lines @@
                         Code:    http.StatusBadRequest,
                 }
         }
         buildNum := p.ByName("build")
         if buildNum == "" {
                 return nil, &miloerror.Error{
                         Message: "No build num",
                         Code:    http.StatusBadRequest,
                 }
         }
+        num, err := strconv.Atoi(buildNum)
+        if err != nil {
+                return nil, &miloerror.Error{
+                        Message: fmt.Sprintf("%s does not look like a number", buildNum),
+                        Code:    http.StatusBadRequest,
+                }
+        }
 
-        result, err := build(c, master, builder, buildNum)
+        result, err := build(c, master, builder, num)
         if err != nil {
                 return nil, err
         }
 
         // Render into the template
         fmt.Fprintf(os.Stderr, "Result: %#v\n\n", result)
         args := &templates.Args{
                 "Build": result,
         }
         return args, nil
@@ skipping 25 matching lines @@
         if err != nil {
                 return nil, err
         }
 
         // Render into the template
         args := &templates.Args{
                 "Builder": result,
         }
         return args, nil
 }