inline manifest viewer (and validator) fails on some manifests

VERIFIED FIXED in 2012-08-23


6 years ago
6 years ago


(Reporter: eviljeff, Assigned: cvan)







6 years ago
The inline manifest viewer is failing to retrieve certain manifests (its returning a 500 error).  Similarly the validator is failing during its daily re-check.

(Despite the similar summary I don't think its related to bug 771101 as there doesn't seem to be any unicode)

URL included is an example, but there are some others as well.

This is the inline viewer url that's 500'ing:

Comment 1

6 years ago

TypeError: SSLError(SSLError(CertificateError("hostname '' doesn't match either of '', ''",),),) is not JSON serializable

Stacktrace (most recent call last):

  File "django/core/handlers/", line 111, in get_response
    response = callback(request, *callback_args, **callback_kwargs)

  File "amo/", line 29, in wrapper
    return func(request, *args, **kw)

  File "amo/", line 61, in wrapper
    return f(request, *args, **kw)

  File "addons/", line 32, in wrapper
    return f(request, addon, *args, **kw)

  File "amo/", line 132, in wrapper

  File "amo/", line 117, in json_response
    response = json.dumps(response)

  File "json/", line 230, in dumps
    return _default_encoder.encode(obj)

  File "json/", line 367, in encode
    chunks = list(self.iterencode(o))

  File "json/", line 309, in _iterencode
    for chunk in self._iterencode_dict(o, markers):

  File "json/", line 275, in _iterencode_dict
    for chunk in self._iterencode(value, markers):

  File "json/", line 317, in _iterencode
    for chunk in self._iterencode_default(o, markers):

  File "json/", line 323, in _iterencode_default
    newobj = self.default(o)

  File "json/", line 344, in default
    raise TypeError(repr(o) + " is not JSON serializable")
Assignee: nobody → cvan
Target Milestone: --- → 2012-08-23

Comment 2

6 years ago
For reference, these are the other two apps that I can find that also fail, where the manifest appears fine when downloaded manually/installed:

There are also other apps that have failed where the host is unreachable, etc.  Having the exact error would be a lot more useful to feed back to the developer.

Comment 3

6 years ago
% curl -I
HTTP/1.1 301 Moved Permanently
Server: nginx/1.3.2
Date: Mon, 20 Aug 2012 16:21:13 GMT
Content-Type: text/html
Content-Length: 184
Connection: keep-alive

% curl -I
HTTP/1.1 200 OK
Content-Type: application/x-web-app-manifest+json
Date: Mon, 20 Aug 2012 16:21:48 GMT
Server: Google Frontend

Are redirects valid?

Comment 4

6 years ago

I'm now ignoring checking SSL certs. Also, if we want to fail on redirects, that's a separate issue.
Last Resolved: 6 years ago
Resolution: --- → FIXED

Comment 5

6 years ago
I filed bug 784727 

But doesn't 500 anymore.
You need to log in before you can comment on or make changes to this bug.