Chromium Code Reviews| 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? |