Closed
Bug 1243274
Opened 9 years ago
Closed 9 years ago
[TV][WebRTC] Open apprtc will show security error
Categories
(Firefox OS Graveyard :: Gaia::TV::Browser, defect, P1)
Tracking
(blocking-b2g:2.5+, b2g-v2.5 fixed, b2g-master fixed)
RESOLVED
FIXED
blocking-b2g | 2.5+ |
People
(Reporter: mlien, Assigned: schien)
References
Details
(Whiteboard: [ft:conndevices][POVB][picked 16/02/01])
Attachments
(3 files)
(deleted),
image/png
|
Details | |
(deleted),
text/x-github-pull-request
|
jj.evelyn
:
review+
jocheng
:
approval-gaia-v2.5+
|
Details |
(deleted),
image/png
|
Details |
[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•9 years ago
|
||
Reporter | ||
Updated•9 years ago
|
Severity: normal → blocker
blocking-b2g: --- → 2.5?
status-b2g-v2.5:
--- → affected
status-b2g-master:
--- → affected
Priority: -- → P1
Comment 2•9 years ago
|
||
Hi ShianYow,
Could you help to find someone who can check this issue?
Flags: needinfo?(shianyow)
Reporter | ||
Updated•9 years ago
|
Blocks: TV_Testing_2.6
Comment 3•9 years ago
|
||
Junior, could you check this issue? Thanks.
Assignee: nobody → juhsu
Flags: needinfo?(shianyow)
Updated•9 years ago
|
Flags: needinfo?(juhsu)
Updated•9 years ago
|
blocking-b2g: 2.5? → 2.5+
Comment 4•9 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•9 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•9 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 | ||
Updated•9 years ago
|
Assignee: juhsu → schien
Group: panasonic-confidential
Component: Gaia::TV → Gaia::TV::Browser
Flags: needinfo?(jocheng)
Comment 7•9 years ago
|
||
Assignee | ||
Comment 8•9 years ago
|
||
Comment on attachment 8713416 [details]
[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)
Assignee | ||
Comment 9•9 years ago
|
||
Comment on attachment 8713416 [details]
[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•9 years ago
|
Attachment #8713416 -
Flags: review?(ehung) → review+
Comment 10•9 years ago
|
||
merged into master:
https://github.com/mozilla-b2g/gaia/commit/e5dedb49b2008202a7fb33a14d58159f731da119
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 11•9 years ago
|
||
Josh: please ask partner to pick this patch, and introduce to their browser app. Thanks.
Flags: needinfo?(jocheng)
Assignee | ||
Comment 13•9 years ago
|
||
Comment on attachment 8713416 [details]
[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•9 years ago
|
||
Comment on attachment 8713416 [details]
[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•9 years ago
|
Comment 15•9 years ago
|
||
with patch to browser as below,
+ "audio-capture":{},
+ "video-capture":{}
erorr message is changed, but not works well.
Is this expected behaviour?
Updated•9 years ago
|
Attachment #8713416 -
Flags: approval‑mozilla‑b2g44+ → approval-gaia-v2.5+
Comment 16•9 years ago
|
||
Hi SC,
Could you check comment 15 from partner? Thanks!
Flags: needinfo?(schien)
Assignee | ||
Comment 17•9 years ago
|
||
The new error message is the expected behavior because Stingray TV doesn't support camera.
Flags: needinfo?(schien)
Comment 18•9 years ago
|
||
Comment 19•9 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.
Description
•