Status

()

Firefox
Screen Sharing Whitelist
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: hthetiot, Assigned: mreavy)

Tracking

Trunk
Firefox 51
All
Other
Points:
---

Firefox Tracking Flags

(firefox49 fixed, firefox50 fixed, firefox51 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
>> Vendor name
Sylaps

>> Point of contact
support@sylaps.com

>> Site URLs
https://sylaps.com
https://*.sylaps.com

>> Does the application provide users with notice and control?
https://sylaps.com/terms

>> 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

Comment 1

2 years ago
Following-up on this mail to get attention as it is important for Qwant business in the next quarter.

Comment 2

2 years ago
+ for going ahead to add to the list Maire.
Flags: needinfo?(mreavy)
(Assignee)

Comment 3

2 years ago
Created attachment 8785447 [details] [diff] [review]
sylaps.patch

Randell -- Can you review and land this? Thanks.
Flags: needinfo?(mreavy)
Attachment #8785447 - Flags: review?(rjesup)
(Assignee)

Updated

2 years ago
Assignee: nobody → mreavy
(Assignee)

Updated

2 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true

Updated

2 years ago
Attachment #8785447 - Flags: review?(rjesup) → review+
(Assignee)

Comment 4

2 years ago
Comment on attachment 8785447 [details] [diff] [review]
sylaps.patch

Approval Request Comment
[Feature/regressing bug #]: screensharing whitelist 
[User impact if declined]: Sylaps/Quant needs their domain added to the whitelist to allow screensharing and to avoid having users modify settings in about: config.  They are deploying their service, and they'd like it to work with Firefox so they can promote it working in Firefox.
[Describe test coverage new/current, TreeHerder]: manually verified (it's a text only change)
[Risks and why]: Extremely low risk, very easily verified, pref only change.  It would help Sylaps/Quant tremendously to have this in Firefox 49 to coincide with their new features/service.  They are launching in Q4 and want to promote their product/service working in Firefox.
[String/UUID change made/needed]: No strings
Attachment #8785447 - Flags: approval-mozilla-beta?
Attachment #8785447 - Flags: approval-mozilla-aurora?

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f18100fdf2ae
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51

Comment 7

2 years ago
I can see target milestone is Firefox 51, we talked about Firefox 50. Any chance to get this fix in Firefox 50 at the latest ?

Comment 8

2 years ago
Comment on attachment 8785447 [details] [diff] [review]
sylaps.patch

Makes sense, Aurora50+
Attachment #8785447 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Updated

2 years ago
status-firefox49: --- → affected
status-firefox50: --- → affected

Comment 9

2 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/9f9819fd9c30
status-firefox50: affected → fixed
Comment on attachment 8785447 [details] [diff] [review]
sylaps.patch

Fine for beta uplift as well, adding support for some domains to the whitelist.
Attachment #8785447 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Comment 11

2 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/f145c179fc1a
status-firefox49: affected → fixed
You need to log in before you can comment on or make changes to this bug.