Closed
Bug 1126952
Opened 11 years ago
Closed 10 years ago
Enable screensharing in whitelist for TokBox
Categories
(Firefox Graveyard :: Screen Sharing Whitelist, defect)
Tracking
(firefox36 fixed, firefox37 fixed, firefox38 fixed)
RESOLVED
FIXED
Firefox 38
People
(Reporter: msander, Assigned: mreavy)
References
Details
Attachments
(1 file)
2.01 KB,
patch
|
jesup
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
>> Vendor name
TokBox
>> Point of contact
Michael Sander
>> Site URLs
*.tokbox.com
>> Does the application provide users with notice and control?
https://tokbox.com/support/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
Assignee | ||
Comment 2•11 years ago
|
||
Hi Michael -- Do you need "tokbox.com" to be whitelisted as well or just "*.tokbox.com"?
Flags: needinfo?(msander)
Assignee | ||
Comment 4•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → mreavy
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Assignee | ||
Comment 5•11 years ago
|
||
Chad/Brad -- Jishnu is deferring to you on whitelist decisions. Can I get approval from either of you to add TokBox to the Firefox screensharing whitelist? My plan is to land it on Nightly and then ask for uplift to Aurora and Beta (if there's still time to get it into Beta) since it's a trivial change. Thanks.
Flags: needinfo?(cweiner)
Flags: needinfo?(blassey.bugs)
Assignee | ||
Comment 6•11 years ago
|
||
Comment on attachment 8559213 [details] [diff] [review]
tokbox.com added to screensharing whitelist
Review of attachment 8559213 [details] [diff] [review]:
-----------------------------------------------------------------
Randell -- can you sanity check (especially for stupid typos) and land this today? Thanks.
Attachment #8559213 -
Flags: review?(rjesup)
Assignee | ||
Comment 7•11 years ago
|
||
I got approval from Chad via email a few minutes ago. We should be able to get this into tomorrow's Nightly. Then I'll ask for aurora/beta approval.
Flags: needinfo?(cweiner)
Flags: needinfo?(blassey.bugs)
Updated•11 years ago
|
Attachment #8559213 -
Flags: review?(rjesup) → review+
Updated•11 years ago
|
Summary: TokBox → Enable screensharing in whitelist for TokBox
Comment 8•11 years ago
|
||
Comment 9•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox38:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 38
Assignee | ||
Comment 10•10 years ago
|
||
Comment on attachment 8559213 [details] [diff] [review]
tokbox.com added to screensharing whitelist
Approval Request Comment
[Feature/regressing bug #]:screensharing whitelist
[User impact if declined]: screensharing won't work with tokbox.com unless the user adds it to the whitelist or installs a special extension
[Describe test coverage new/current, TreeHerder]: confirmed visually (text change)
[Risks and why]: Very low risk. It's a text change to a whitelist that is easily verified if it's working or not
[String/UUID change made/needed]: None
Attachment #8559213 -
Flags: approval-mozilla-beta?
Attachment #8559213 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
status-firefox36:
--- → affected
status-firefox37:
--- → affected
Updated•10 years ago
|
Attachment #8559213 -
Flags: approval-mozilla-beta?
Attachment #8559213 -
Flags: approval-mozilla-beta+
Attachment #8559213 -
Flags: approval-mozilla-aurora?
Attachment #8559213 -
Flags: approval-mozilla-aurora+
Comment 11•10 years ago
|
||
Comment 12•10 years ago
|
||
Updated•7 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•