Cannot submit an updated packaged app with the same origin as the old version of the app

VERIFIED FIXED in 2013-07-18

Status

P1
major
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: jsmith, Assigned: robhudson)

Tracking

2013-07-18
Points:
---

Details

(Whiteboard: [qa+])

(Reporter)

Description

5 years ago
STR

1. Submit a privileged app to marketplace with a valid origin (e.g. app://test.webapi.permissions.com)
2. Publish that app
3. Go to Manage Status & Versions
4. Under upload new version, try to upload a privileged app with a new version that contains the same origin as #1

Expected

The app should be uploaded successfully.

Actual

The app fails to be uploaded successfully with an error saying - An app already exists on this domain; only one app per domain is allowed. This isn't right - if an app specifies an origin in their manifest that matches the old version of the app, then it should be allowed to be uploaded.
(Assignee)

Comment 1

5 years ago
P1 b/c it's blocking QA.

We need to exclude the app in our db query when checking for existing origins.
Assignee: nobody → robhudson.mozbugs
Priority: -- → P1
(Assignee)

Comment 2

5 years ago
https://github.com/mozilla/zamboni/commit/c72ff7f
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-07-18

Comment 3

5 years ago
Issue is still reproducing in https://marketplace-dev.allizom.org/developers/ on FF25 (Win 7).
Please see screencast http://screencast.com/t/qfpT12cZA
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 4

5 years ago
Thanks for catching that. Fixed now.
https://github.com/mozilla/zamboni/commit/3498f75
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
(Assignee)

Updated

5 years ago
Whiteboard: [qa+]
(Reporter)

Comment 5

5 years ago
I need to verify a platform bug in relation to this, so I'll test this by caveat as well on marketplace dev to do two verifications at once.
Keywords: verifyme
QA Contact: jsmith

Comment 6

5 years ago
Still doesn't work. See http://cl.ly/image/3g3J0I373X2X
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Reporter)

Updated

5 years ago
Keywords: verifyme
QA Contact: jsmith

Comment 7

5 years ago
whoops, wrong bug
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Keywords: verifyme
QA Contact: jsmith
(Reporter)

Comment 8

5 years ago
lgtm - my test app I was using that was a privileged packaged app with an origin specified updating to a privileged packaged app with the same origin was successfully able to be uploaded to marketplace.

I did notice bug 895708 by caveat however testing the update path, but that's likely a different issue.
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.