add String method to Request

data.go

@@ -186,6 +186,14 @@ type Request struct {
 	Options Options  // Image transformation to perform
 }
 
+// String returns the request URL as a string, with r.Options encoded in the
+// URL fragment.
+func (r Request) String() string {
+	u := *r.URL
+	u.Fragment = r.Options.String()
+	return u.String()
+}
+
 // NewRequest parses an http.Request into an imageproxy Request.  Options and
 // the remote image URL are specified in the request path, formatted as:
 // /{options}/{remote_url}.  Options may be omitted, so a request path may

imageproxy.go

@@ -95,12 +95,7 @@ func (p *Proxy) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	u := req.URL.String()
+	resp, err := p.Client.Get(req.String())
-	if req.Options != emptyOptions {
-		u += "#" + req.Options.String()
-	}
-
-	resp, err := p.Client.Get(u)
 	if err != nil {
 		msg := fmt.Sprintf("error fetching remote image: %v", err)
 		glog.Error(msg)