code review 4893043: url: new package

src/pkg/http/cgi/host.go

 // Copyright 2011 The Go Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file.
 
 // This file implements the host side of CGI (being the webserver
 // parent process).
 
 // Package cgi implements CGI (Common Gateway Interface) as specified
 // in RFC 3875.
 //
@@ skipping 258 matching lines @@
 
 func (h *Handler) printf(format string, v ...interface{}) {
         if h.Logger != nil {
                 h.Logger.Printf(format, v...)
         } else {
                 log.Printf(format, v...)
         }
 }
 
 func (h *Handler) handleInternalRedirect(rw http.ResponseWriter, req *http.Request, path string) {
-        url, err := req.URL.ParseURL(path)
+        url, err := req.URL.Parse(path)
         if err != nil {
                 rw.WriteHeader(http.StatusInternalServerError)
                 h.printf("cgi: error resolving local URI path %q: %v", path, err)
                 return
         }
         // TODO: RFC 3875 isn't clear if only GET is supported, but it
         // suggests so: "Note that any message-body attached to the
         // request (such as for a POST request) may not be available
         // to the resource that is the target of the redirect."  We
         // should do some tests against Apache to see how it handles
@@ skipping 24 matching lines @@
                 return '_'
         case rune == '=':
                 // Maybe not part of the CGI 'spec' but would mess up
                 // the environment in any case, as Go represents the
                 // environment as a slice of "key=value" strings.
                 return '_'
         }
         // TODO: other transformations in spec or practice?
         return rune
 }