Last modified: 2014-09-08 15:13:00 UTC
Found in browser test build https://integration.wikimedia.org/ci/job/browsertests-Echo-test2.wikipedia.org-linux-chrome-sauce/11/console Varnish yield a 500 (or 503) error which has a HTML payload. The response object is passed to json decode which should not happen on a 500. Trace: 757: unexpected token at '<!DOCTYPE html> <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en"> ... </html>' (JSON::ParserError) gems/json-1.8.1/lib/json/common.rb:155:in `parse' gems/json-1.8.1/lib/json/common.rb:155:in `parse' gems/mediawiki_api-0.2.1/lib/mediawiki_api/response.rb:85:in `response_object' gems/mediawiki_api-0.2.1/lib/mediawiki_api/response.rb:46:in `data' gems/mediawiki_api-0.2.1/lib/mediawiki_api/client.rb:86:in `log_in' gems/mediawiki_selenium-0.3.2/lib/mediawiki_selenium/support/pages/api_page.rb:14:in `client' Echo/tests/browser/features/step_definitions/common_steps.rb:32:in `block in <top (required)>'
Change 157300 had a related patch set uploaded by Dduvall: Error handling for HTTP 4xx and 5xx responses https://gerrit.wikimedia.org/r/157300
Change 157300 merged by jenkins-bot: Error handling for HTTP 4xx and 5xx responses https://gerrit.wikimedia.org/r/157300
Dan, can this bug now be resolved?
The fix has been merged. Do we usually wait to mark bugs fixed upon a new release, or is that sufficient?
This should do it