Closed
Bug 1137057
Opened 9 years ago
Closed 9 years ago
Interactive Intelligence
Categories
(Firefox Graveyard :: Screen Sharing Whitelist, defect)
Tracking
(firefox39 fixed, firefox40 fixed, firefox41 fixed)
RESOLVED
FIXED
Firefox 41
People
(Reporter: xander, Assigned: mreavy)
Details
Attachments
(3 files, 1 obsolete file)
802.95 KB,
image/png
|
Details | |
76.60 KB,
image/png
|
Details | |
2.87 KB,
patch
|
jesup
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
>> Vendor name Interactive Intelligence >> Point of contact Xander Dumaine >> Site URLs mypurecloud.com, mypurecloud.com.au >> Does the application provide users with notice and control? http://help.mypurecloud.com/articles/purecloud-privacy-policy/ >> Have you seen and agree with the Developer Screen Sharing Submission policy? Yes >> Does the usage comply with the Developer Screen Sharing Submission Policy guidelines? Yes
Reporter | ||
Comment 1•9 years ago
|
||
Currently the actual apps live at https://apps.mypurecloud.com/directory/ - we'll be launching screen sharing as a feature soon (already live internally), and would like to be able to list Firefox as add-on and plugin free. We're currently using a published add-on to alter the allowed domains list, but would love to be whitelisted instead.
Reporter | ||
Comment 2•9 years ago
|
||
Could I get an update on this? The app is now live, and free to use. Users can sign up for an account at http://www.inin.com/pages/signup.aspx and begin using WebRTC video chat with screen sharing.
Reporter | ||
Comment 3•9 years ago
|
||
Bump.
Updated•9 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(mreavy)
Assignee | ||
Comment 4•9 years ago
|
||
Sorry for the delay on this. I've talked with Shell, and we'll resolve this bug within the week.
Flags: needinfo?(mreavy)
Comment 5•9 years ago
|
||
Hi Maire - based on our review, we can add this request to into the whitelist on nightly (currently Fx41). Likely should put in for uplift based on timing request for their live application. Hi Xander, Sorry for the delay and thank you for your patience. Your privacy policy looks good, but is hard to locate. Could that be moved somewhere more accessible for users?
Flags: needinfo?(xander.dumaine)
Flags: needinfo?(mreavy)
Reporter | ||
Comment 6•9 years ago
|
||
There's a link to the privacy policy on the login screen, and another in the help menu always visible.
Flags: needinfo?(xander.dumaine)
Reporter | ||
Comment 7•9 years ago
|
||
There's a link to the privacy policy on the login screen, and another in the help menu always visible.
Reporter | ||
Comment 8•9 years ago
|
||
Just added two screenshots of the ways to access it. These were the places legal and UX worked on for placement. It's also searchable from help. Do you think we need more than that?
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(sescalante)
Assignee | ||
Comment 9•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → mreavy
Status: NEW → ASSIGNED
Assignee | ||
Comment 10•9 years ago
|
||
Comment on attachment 8610576 [details] [diff] [review] Interactive Intelligence added to screensharing whitelist Review of attachment 8610576 [details] [diff] [review]: ----------------------------------------------------------------- Randell, please sanity check and land this. NOTE: We also have approval to land Bug 1151351, which I'll be updating next. So it probably makes sense to land those two together today. Thanks!
Attachment #8610576 -
Flags: review?(rjesup)
Assignee | ||
Comment 11•9 years ago
|
||
(In reply to Xander Dumaine from comment #8) > Just added two screenshots of the ways to access it. These were the places > legal and UX worked on for placement. It's also searchable from help. Do you > think we need more than that? Thanks, Xander! This is sufficient. If you could take a look at the patch on this bug to verify that your domain names are listed as you need them to be, that'd be helpful. My plan is to land the patch today and then request uplift (to Fx40 and Fx39) once it reaches Nightly (Fx41) tomorrow.
Flags: needinfo?(xander.dumaine)
Flags: needinfo?(sescalante)
Flags: needinfo?(mreavy)
Reporter | ||
Comment 12•9 years ago
|
||
The domains look right, but are missing the "*." prefix for wildcard subdomains.
Flags: needinfo?(xander.dumaine) → needinfo?(mreavy)
Reporter | ||
Comment 13•9 years ago
|
||
That would be my fault for not including that in the original request - sorry about that.
Assignee | ||
Comment 14•9 years ago
|
||
(In reply to Xander Dumaine from comment #12) > The domains look right, but are missing the "*." prefix for wildcard subdomains. Do you need just *.mypurecloud.com, *.mypurecloud.com.au? Or all 4: mypurecloud.com, mypurecloud.com.au, *.mypurecloud.com, *.mypurecloud.com.au
Flags: needinfo?(mreavy) → needinfo?(xander.dumaine)
Reporter | ||
Comment 15•9 years ago
|
||
(In reply to Maire Reavy [:mreavy] (Plz needinfo me) from comment #14) > Do you need just *.mypurecloud.com, *.mypurecloud.com.au? Or all 4: > mypurecloud.com, mypurecloud.com.au, *.mypurecloud.com, *.mypurecloud.com.au I do not need the bare root domains, just *.mypurecloud.com, *.mypurecloud.com.au
Flags: needinfo?(xander.dumaine)
Assignee | ||
Comment 16•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8610576 -
Attachment is obsolete: true
Attachment #8610576 -
Flags: review?(rjesup)
Assignee | ||
Comment 17•9 years ago
|
||
Comment on attachment 8610606 [details] [diff] [review] Interactive Intelligence added to screensharing whitelist Review of attachment 8610606 [details] [diff] [review]: ----------------------------------------------------------------- Randell -- This has the changes that Xander needs. Please review and land this patch. I'll also be updating the patch on Bug 1151351 since it was based on the older patch of this bug that I just obsoleted. Thanks.
Attachment #8610606 -
Flags: review?(rjesup)
Reporter | ||
Comment 18•9 years ago
|
||
Looks good to me. Thanks!
Assignee | ||
Comment 19•9 years ago
|
||
(In reply to Xander Dumaine from comment #18) > Looks good to me. Thanks! Cool. We'll land it today.
Updated•9 years ago
|
Attachment #8610606 -
Flags: review?(rjesup) → review+
https://hg.mozilla.org/mozilla-central/rev/d8b2b0ce0e52
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox41:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Assignee | ||
Comment 22•9 years ago
|
||
Comment on attachment 8610606 [details] [diff] [review] Interactive Intelligence added to screensharing whitelist Approval Request Comment [Feature/regressing bug #]:screensharing whitelist [User impact if declined]: Interactive Intelligence will be launching screensharing very soon, and they need their domain added to our whitelist so that screensharing works for all Firefox users when they launch. [Describe test coverage new/current, TreeHerder]: manually verified (it's a text only change) [Risks and why]: Extremely low risk, very easily verified [String/UUID change made/needed]: No strings
Attachment #8610606 -
Flags: approval-mozilla-beta?
Attachment #8610606 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
status-firefox39:
--- → affected
status-firefox40:
--- → affected
Comment 23•9 years ago
|
||
Comment on attachment 8610606 [details] [diff] [review] Interactive Intelligence added to screensharing whitelist Low risk for the release, ok to uplift.
Attachment #8610606 -
Flags: approval-mozilla-beta?
Attachment #8610606 -
Flags: approval-mozilla-beta+
Attachment #8610606 -
Flags: approval-mozilla-aurora?
Attachment #8610606 -
Flags: approval-mozilla-aurora+
Updated•6 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•