Cannot launch pop-up dialog for facebook and browser ID when running app natively

VERIFIED FIXED

Status

Firefox Graveyard
Webapp Runtime
--
critical
VERIFIED FIXED
5 years ago
a year ago

People

(Reporter: jsmith, Assigned: Mardak)

Tracking

unspecified
Bug Flags:
in-moztrap -

Details

(Whiteboard: [topapps], [marketplace-beta+])

Attachments

(1 attachment, 1 obsolete attachment)

1.07 KB, patch
Details | Diff | Splinter Review
(Reporter)

Description

5 years ago
Steps:

1. Install the latest build of the extension
2. Go to apps-preview.mozilla.org
3. Install "Forces of War" natively
4. Run Forces of War natively
5. Select the facebook login button to login

Expected:

A pop-up dialog box should appear allowing the user to login via facebook.

Actual:

No pop-up appears.
(Reporter)

Updated

5 years ago
Whiteboard: [topapps]
(Reporter)

Updated

5 years ago
Whiteboard: [topapps] → [topapps] qa-wanted
(Reporter)

Updated

5 years ago
Duplicate of this bug: 732628
(Reporter)

Updated

5 years ago
Whiteboard: [topapps] qa-wanted → [topapps] qa-wanted [marketplace-beta?]

Updated

5 years ago
Keywords: qawanted
Whiteboard: [topapps] qa-wanted [marketplace-beta?] → [topapps], [marketplace-beta?]
(Reporter)

Updated

5 years ago
Component: Extension → Desktop Runtime
QA Contact: extension → desktop-runtime
(Reporter)

Updated

5 years ago
Blocks: 738816
(Reporter)

Updated

5 years ago
No longer blocks: 738816
(Reporter)

Comment 2

5 years ago
Confirmed to still be happening from testing the marketplace open web application.
Keywords: qawanted
(Assignee)

Updated

5 years ago
Blocks: 746229
(Reporter)

Comment 3

5 years ago
This is definitely a problem with pretty much anything that does integration to external origin. Common use case is anything that has pop-ups, including:

- Persona Integration
- Clicking facebook, google plus like buttons
- JavaScript pop-ups

Moving to critical, given that this greatly causes problems in functionality from a user's perspective, as many applications do use pop-ups.
Severity: normal → critical
(Reporter)

Updated

5 years ago
Duplicate of this bug: 746229
(Reporter)

Updated

5 years ago
No longer blocks: 746229
I need to know how many apps this will affect. If it's a large number of partner apps then I need to speak with BD to get their take on it.
(Reporter)

Comment 6

5 years ago
F
Keywords: qawanted
(Reporter)

Comment 7

5 years ago
Investigation needed on the following applications:

- Backpack
- Basecamp
- Campfire
- Highrise
- Ta-Da List
- Lord of Ultima
- Soundcloud
- Times Crossword
- Sky Guide
(Reporter)

Updated

5 years ago
Blocks: 737571
(Reporter)

Comment 8

5 years ago
Per a discussion with Kumar, this will prevent in-app payments from working correctly in the desktop WebRT, given that they rely on logging in with browser ID, which uses a pop-up.
(Reporter)

Updated

5 years ago
Whiteboard: [topapps], [marketplace-beta?] → [topapps], [marketplace-beta]
(Reporter)

Comment 9

5 years ago
Time Crossword also relies on persona integration. Without this fixed, this application has limited functionality. Switching this to block marketplace beta.
Keywords: qawanted
(Reporter)

Comment 10

5 years ago
This stops account management also from working in Soundcloud.
(Reporter)

Updated

5 years ago
Whiteboard: [topapps], [marketplace-beta] → [topapps], [marketplace-beta?]
(Reporter)

Comment 11

5 years ago
Jen to follow up with BD on this bug.
From my perspective having looked at around 60 submissions and on the marketplace product side of things:

* I've seen around 10 submissions affected by this, where primary functionality of an app is unusable without Facebook Connect pop-up (many of these are BD-sourced apps)

* one of our important goals for this milestone was getting feedback on in-app purchases. We did a ton of work to hustle and get in-app payments ready for this release, and we're thwarted by the BrowserID pop-up not being able to open.

If this can't be fixed more generically, a temporary solution for FBConnect and BrowserID would solve most of the problems I've run into.
(Reporter)

Updated

5 years ago
Whiteboard: [topapps], [marketplace-beta?] → [topapps], [marketplace-beta+]
(Assignee)

Comment 13

5 years ago
Created attachment 618751 [details] [diff] [review]
v1

Builds on bug 707836 for controlling navigation and bug 732631 for allowing popups to explicitly allow https://browserid.org navigation in a popup.
Assignee: nobody → edilee
Status: NEW → ASSIGNED
(Reporter)

Comment 14

5 years ago
(In reply to Edward Lee :Mardak from comment #13)
> Created attachment 618751 [details] [diff] [review]
> v1
> 
> Builds on bug 707836 for controlling navigation and bug 732631 for allowing
> popups to explicitly allow https://browserid.org navigation in a popup.

What about facebook connect? That's very common in web apps, especially top apps. Also, how will we address this in the long run? An app probably should be able to control the "allowed_origins" parameter (bug 741955).
(Assignee)

Comment 15

5 years ago
(In reply to Jason Smith from comment #14)
> how will we address this in the long run?
In the long run, at least for Persona, is to implement navigator.id.. that still doesn't address the issue of facebook connect though.
Comment on attachment 618751 [details] [diff] [review]
v1

r=myk with Facebook Connect's origin added to this list and a bug filed to implement an manifest-specified list of allowed origins!
Attachment #618751 - Flags: review-
Comment on attachment 618751 [details] [diff] [review]
v1

Erm, I'm going to make this r+ with conditions noted in comment 16 so you can carry forward review and land the patch with Facebook Connect added to the list.
Attachment #618751 - Flags: review- → review+
Erm, and I've gone ahead and filed the followup bug (bug 749415).
(Assignee)

Comment 19

5 years ago
Created attachment 618849 [details] [diff] [review]
for checkin

Explicitly allow Persona https://browserid.org and Facebook Connect https://www.facebook.com for now.
Attachment #618751 - Attachment is obsolete: true
(Assignee)

Comment 20

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/94f63dd34a0d
https://hg.mozilla.org/mozilla-central/rev/94f63dd34a0d
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Reporter)

Comment 22

5 years ago
(In reply to Ms2ger from comment #21)
> https://hg.mozilla.org/mozilla-central/rev/94f63dd34a0d

Is this is really resolved fixed in this case? The fix only addresses pop-ups with facebook and browser ID.
If you need the bug to remain open after its patch lands, put "[leave open]" on the whiteboard.
(Reporter)

Updated

5 years ago
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [topapps], [marketplace-beta+] → [topapps], [marketplace-beta+], [leave open]
(Reporter)

Updated

5 years ago
Keywords: qawanted
(Reporter)

Comment 24

5 years ago
Flagging qawanted to test the two origins added.
(Reporter)

Updated

5 years ago
Whiteboard: [topapps], [marketplace-beta+], [leave open] → [topapps], [marketplace-beta?], [leave open]
(Assignee)

Comment 25

5 years ago
We can close this for now. Myk has already filed a followup bug 749415.
Blocks: 749415
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
(Reporter)

Updated

5 years ago
Whiteboard: [topapps], [marketplace-beta?], [leave open] → [topapps], [marketplace-beta?]
(Reporter)

Updated

5 years ago
Whiteboard: [topapps], [marketplace-beta?] → [topapps], [marketplace-beta+]
(Reporter)

Updated

5 years ago
Keywords: qawanted
Summary: Cannot launch pop-up dialog not on the same origin of the app when running app natively → Cannot launch pop-up dialog for facebook and browser ID when running app natively
(Reporter)

Comment 26

5 years ago
Okay. Updated the bug title, so that the fix reflects the name of the bug for tracking purposes.
(Reporter)

Comment 27

5 years ago
Verified for browser ID pop-up.
(Reporter)

Comment 28

5 years ago
Verified with facebook login.
Status: RESOLVED → VERIFIED
(Reporter)

Comment 29

5 years ago
Moving this back to resolved fixed. There's a test case in question to reopen this bug, involving the facebook like button:

Example Test Case Steps:

1. Install Sandglaz (https://marketplace-dev.allizom.org/en-US/app/sandglaz-1/)
2. Launch that application
3. Click the facebook like button on the bottom of the application

Expected:

A application pop-up should appear for www.facebook.com login.

Actual:

The default browser starts. Logging in will result in going to page of nothingness, probably cause it's trying to redirect back to the application.

Additional Notes:

Similar issue occurs with google plus +1 button, although it's we don't have that whitelisted yet.
Status: VERIFIED → RESOLVED
Last Resolved: 5 years ago5 years ago
(Reporter)

Comment 30

5 years ago
Moving this back to verified per an email from Myk. Will open a separate bug to track the facebook like button and google +1 button issue.
Status: RESOLVED → VERIFIED
(Reporter)

Updated

5 years ago
No longer blocks: 737571
Component: Desktop Runtime → Webapp Runtime
Product: Web Apps → Firefox
(Reporter)

Updated

5 years ago
Flags: in-moztrap-
(Reporter)

Updated

5 years ago
QA Contact: desktop-runtime → jsmith
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.