887034 - Marketplace packaged app is not possible to update due to incorrect app store ID

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Jason Smith [:jsmith]]

The marketplace packaged app isn't possible to update on Firefox OS 1.1 due to incorrect store ID in the signing portion of the zip file. As such, users will not be able to update the marketplace packaged app on Firefox OS 1.1 right now. We need to update the marketplace packaged app zip file on device with the correct store ID in the ids.json file.

Comment 1

[Christopher Van Wiemeersch [:cvan]]

Marketplace App in Gaia:
.zip: https://github.com/mozilla-b2g/gaia/blob/master/external-apps/marketplace.firefox.com/application.zip
ids.json: http://cl.ly/image/242x0i1D0O0L/Screen%20Shot%202013-06-25%20at%203.56.49%20PM.png

Latest Marketplace packaged app version:
.zip: https://marketplace.firefox.com/downloads/file/213482/marketplace-20130613140040.zip
ids.json: http://f.cl.ly/items/3S0h2G0e030W0p330t0b/Screen%20Shot%202013-06-25%20at%204.34.38%20PM.png

"8af8c763-da9b-444d-a911-206f9e225b55" is the same "id" in both. I'm not clear what needs changing - can someone confirm?

Comment 2

[Jason Smith [:jsmith]]

Followed up with Chris in IRC about this - the id values are actually different, which is causing this bug. We just need a pull request to update the marketplace packaged app with the correct ID.

Comment 3

[Wayne Chang [:wchang]]

Triage - leo+ as this must be fixed.

Comment 4

[Christopher Van Wiemeersch [:cvan]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/10645

Comment 5

[Jason Smith [:jsmith]]

Comment on attachment 767983 [details]
https://github.com/mozilla-b2g/gaia/pull/10645

Looks okay to me. I'll mark review on Julien for landing.

FYI - Fabrice is out until June 29th.

Comment 6

[Julien Wajsberg [:julienw]]

Comment on attachment 767983 [details]
https://github.com/mozilla-b2g/gaia/pull/10645

lgtm
r=me

Comment 7

[Julien Wajsberg [:julienw]]

master: 084d7a34f1678b5e25233267607bb00db549b39a

Comment 8

[Gareth Aye [:gaye] (back from PTO)]

I was not able to uplift this bug to v1-train.  If this bug has dependencies which are not marked in this bug, please comment on this bug.  If this bug depends on patches that aren't approved for v1-train, we need to re-evaluate the approval.  Otherwise, if this is just a merge conflict, you might be able to resolve it with:

  git checkout v1-train
  git cherry-pick -x -m1 084d7a34f1678b5e25233267607bb00db549b39a
  <RESOLVE MERGE CONFLICTS>
  git commit

Comment 9

[Julien Wajsberg [:julienw]]

taking the uplift

Comment 10

[Julien Wajsberg [:julienw]]

so, there was a conflict because bug 875849 was not uplifted after all.

v1-train: 50a1dec42e154be9c3e140120bb06ac1071b2e4c

Comment 11

[John Ford [:jhford] CET/CEST Berlin Time]

v1.1.0hd: 50a1dec42e154be9c3e140120bb06ac1071b2e4c