Index: chrome/common/extensions/docs/server/chromeextensionsdocs.py |
diff --git a/chrome/common/extensions/docs/server/chromeextensionsdocs.py b/chrome/common/extensions/docs/server/chromeextensionsdocs.py |
index a15de088bc463a5595dcd0e9878ad63e7010e294..db9585552cc1df6b34b046f71d826e36b4e81179 100644 |
--- a/chrome/common/extensions/docs/server/chromeextensionsdocs.py |
+++ b/chrome/common/extensions/docs/server/chromeextensionsdocs.py |
@@ -72,7 +72,9 @@ class MainPage(webapp.RequestHandler): |
logging.info("Path: " + self.request.path) |
logging.info("Url: " + url) |
try: |
- result = urlfetch.fetch(url + self.request.query_string) |
+ # TODO(aa): Do we need to pass the querystring through here? We used to, |
Mihai Parparita -not on Chrome
2011/11/15 14:44:49
You could also switch this to url + '?' + self.req
Aaron Boodman
2011/11/15 19:06:57
Aha! In that case, I'm comfortable just removing t
|
+ # but that caused problems. See crbug.com/104272. |
+ result = urlfetch.fetch(url) |
if result.status_code != 200: |
logging.error("urlfetch failed: " + url) |
# TODO(nickbaum): what should we do when the urlfetch fails? |