All users were logged out of Bugzilla on October 13th, 2018

[TV][WebRTC] Open apprtc will show security error

RESOLVED FIXED

Status

P1
blocker
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mlien, Assigned: schien)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(blocking-b2g:2.5+, b2g-v2.5 fixed, b2g-master fixed)

Details

(Whiteboard: [ft:conndevices][POVB][picked 16/02/01])

Attachments

(3 attachments)

(Reporter)

Description

3 years ago
[Description]
Open apprtc will show security error

[Reproduce Steps]
1. Launch Browser
2. Navigating to "apprtc.appspot.com"
3. Join the room

[Build Information]
Environment: 2.5 Simulator
Gaia: https://github.com/mozilla-b2g/gaia/commit/8a0d9deb2cd47df25750ed2c6820909e4feac3bb
Gecko: https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/2b6f8826861a846c3d1d98ef595c8dfbf8b59328
Mulet: https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/2b6f8826861a846c3d1d98ef595c8dfbf8b59328 

[Expected Result]
Can open the chat room successfully

[Actual Result]
It will show security error
(Reporter)

Comment 1

3 years ago
Created attachment 8712525 [details]
Screenshot from 2016-01-27 13:32:02.png
(Reporter)

Updated

3 years ago
Severity: normal → blocker
blocking-b2g: --- → 2.5?
status-b2g-v2.5: --- → affected
status-b2g-master: --- → affected
Priority: -- → P1
(Reporter)

Updated

3 years ago
Blocks: 1233624

Comment 2

3 years ago
Hi ShianYow,
Could you help to find someone who can check this issue?
Flags: needinfo?(shianyow)
(Reporter)

Updated

3 years ago
Blocks: 1239346

Comment 3

3 years ago
Junior, could you check this issue?  Thanks.
Assignee: nobody → juhsu
Flags: needinfo?(shianyow)

Updated

3 years ago
Flags: needinfo?(juhsu)

Updated

3 years ago
blocking-b2g: 2.5? → 2.5+

Comment 4

3 years ago
I have no idea about the mulet error.
Need some time to figure out, but I guess it's with lower priority for mulet.

For real TV, it causes error since lack of camera and mic. |GetUserMedia| could not success for this website. If we'd like to test streaming for webrtc, it needs another way to investigate.

Therefore, appspot.com is designed to show this message.
Flags: needinfo?(juhsu)

Comment 5

3 years ago
Hello Mike, Josh,
Since it's a P1 blocker,
I'd like to have your suggestion about the next step.
Flags: needinfo?(mlien)
Flags: needinfo?(jocheng)
(Reporter)

Comment 6

3 years ago
Then I'll suggest to disable webrtc or at least disable the streaming channel until we can fix it
Flags: needinfo?(mlien)
Assignee: juhsu → schien
Group: panasonic-confidential
Component: Gaia::TV → Gaia::TV::Browser
Flags: needinfo?(jocheng)
Created attachment 8713416 [details] [review]
[gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master
Comment on attachment 8713416 [details] [review]
[gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master

missing permission in TV browser app for accessing microphone and camera.
Attachment #8713416 - Flags: review?(yliao)
Comment on attachment 8713416 [details] [review]
[gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master

@evelyn can you take a look at this patch, since @yifan is on PTO.
Attachment #8713416 - Flags: review?(yliao) → review?(ehung)

Updated

3 years ago
Attachment #8713416 - Flags: review?(ehung) → review+

Comment 10

3 years ago
merged into master:

https://github.com/mozilla-b2g/gaia/commit/e5dedb49b2008202a7fb33a14d58159f731da119
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Comment 11

3 years ago
Josh: please ask partner to pick this patch, and introduce to their browser app. Thanks.
Flags: needinfo?(jocheng)

Comment 12

3 years ago
SC: you need to a? your patch for uplift. Thanks.
Flags: needinfo?(schien)
Comment on attachment 8713416 [details] [review]
[gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): no
User impact if declined: cannot use GetUserMedia WebAPI in TV browser
Testing completed: manual test
Risk to taking this patch (and alternatives if risky): no risk
String or UUID changes made by this patch: no
Flags: needinfo?(schien)
Attachment #8713416 - Flags: approval‑mozilla‑b2g44?

Comment 14

3 years ago
Comment on attachment 8713416 [details] [review]
[gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master

Approve for TV 2.5
Flags: needinfo?(jocheng)
Attachment #8713416 - Flags: approval‑mozilla‑b2g44? → approval‑mozilla‑b2g44+

Updated

3 years ago
status-b2g-master: affected → fixed

Comment 15

3 years ago
Created attachment 8713587 [details]
apprtc_err.png

with patch to browser as below,
+    "audio-capture":{},
+    "video-capture":{}

erorr message is changed, but not works well.

Is this expected behaviour?

Updated

3 years ago
Attachment #8713416 - Flags: approval‑mozilla‑b2g44+ → approval-gaia-v2.5+

Comment 16

3 years ago
Hi SC,
Could you check comment 15 from partner? Thanks!
Flags: needinfo?(schien)
The new error message is the expected behavior because Stingray TV doesn't support camera.
Flags: needinfo?(schien)

Comment 19

3 years ago
partner side introduced this patch for V2.5 (16/2/1)
Whiteboard: [ft:conndevices][POVB] → [ft:conndevices][POVB][picked 16/02/01]
You need to log in before you can comment on or make changes to this bug.