1012448 - public mini-manifest for packaged app is empty {}

Status: RESOLVED FIXED

(Marketplace Graveyard :: General, defect, P2)

Comment 1

[Lourdes Castillo [:lourcastillo]]

I saw on Firefox marketplace that the manifest's link is bad. https://marketplace.firefox.com/app/54403c95-2f64-4c4a-a0e3-37dedc1f224a/manifest.webapp (the manifest is invalid) packaged https://marketplace.firefox.com/app/transantiagoapp/

Comment 2

[Pablo Sepulveda]

The META-INF folder (with ID) is created by marketplace server. The INVALID_MANIFEST error (by documentation) is by marketplace server, because the manifest.webapp JSON is clean in the server.

Comment 3

[Pablo Sepulveda]

The bug will be reviewed by Marketplace Senior Reviewer.

Comment 4

[Andrew Williamson [:eviljeff]]

this is happening to https://marketplace.firefox.com/app/magic-card/ also
https://marketplace.firefox.com/app/16d6a291-8d91-463a-bc32-b3b746b13b18/manifest.webapp

Comment 5

[Andy McKay]

I see a valid mini manifest, it works for clouserw and andym.

Comment 6

[Andrew Williamson [:eviljeff]]

(In reply to Andy McKay [:andym] from comment #5)
> I see a valid mini manifest, it works for clouserw and andym.

yep same here today, maybe something got fixed in the push.

Comment 7

[Andrew Williamson [:eviljeff]]

possible resurgence with a different app - bug 1016475

Comment 9

[Rob Hudson [:robhudson]]

Reopening...

This seems like it can happen when `get_cached_manifest` is called when there is not approved version. When a version is approved it should get updated but it appears that sometimes it doesn't. We need to find and fix that.

We can also remove this conditional to avoid caching '{}':
https://github.com/mozilla/zamboni/blob/master/mkt/webapps/models.py#L900

Comment 10

[Rob Hudson [:robhudson]]

https://github.com/mozilla/zamboni/commit/5825f51 

This commit avoids setting the cached minifest to '{}' so when a valid version does come into play everything should get set and cached correctly w/o having to track down where to call `force=True`.

Flagging as qa- as it's not clear how to create an app in this state.