diff --git a/api/agent/agent.go b/api/agent/agent.go index 14671f79b..0ec318297 100644 --- a/api/agent/agent.go +++ b/api/agent/agent.go @@ -38,7 +38,6 @@ import ( // TODO all Datastore methods need to take unit of tenancy (app or route) at least (e.g. not just call id) // TODO limit the request body length when making calls // TODO discuss concrete policy for hot launch or timeout / timeout vs time left -// TODO can we get rid of 'GET url?payload' weirdness? // TODO call env need to be map[string][]string to match headers behavior... // TODO it may be nice to have an interchange type for Dispatch that can have // all the info we need to build e.g. http req, grpc req, json, etc. so that diff --git a/api/server/runner.go b/api/server/runner.go index b18fb2458..a36740a7e 100644 --- a/api/server/runner.go +++ b/api/server/runner.go @@ -3,7 +3,6 @@ package server import ( "bytes" "context" - "io/ioutil" "net/http" "path" "strings" @@ -27,12 +26,6 @@ func (s *Server) handleRequest(c *gin.Context) { ctx := c.Request.Context() - if c.Request.Method == "GET" { - // TODO we _could_ check the normal body, this is still weird - // TODO do we need to flush the original body if we do this? (hint: yes) - c.Request.Body = ioutil.NopCloser(strings.NewReader(c.Request.URL.Query().Get("payload"))) - } - r, routeExists := c.Get(api.Path) if !routeExists { r = "/"