Merge lp:~justinmcp/media-hub/fix-allowed-uri-schemes into lp:media-hub

Proposed by David Barth
Status: Superseded
Proposed branch: lp:~justinmcp/media-hub/fix-allowed-uri-schemes
Merge into: lp:media-hub
Diff against target: 14 lines (+3/-1)
1 file modified
src/core/media/apparmor/ubuntu.cpp (+3/-1)
To merge this branch: bzr merge lp:~justinmcp/media-hub/fix-allowed-uri-schemes
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Jim Hodapp (community) code Approve
Review via email: mp+268335@code.launchpad.net

This proposal has been superseded by a proposal from 2015-08-18.

Commit message

Allow media-hub to process streams coming via HTTPS urls

Description of the change

Allow media-hub to process streams coming via HTTPS urls

To post a comment you must log in.
Revision history for this message
Jim Hodapp (jhodapp) wrote :

Looks good, thanks.

review: Approve (code)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Unmerged revisions

151. By Justin McPherson

Add https to allowed URI schemes

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/core/media/apparmor/ubuntu.cpp'
2--- src/core/media/apparmor/ubuntu.cpp 2014-11-26 16:23:19 +0000
3+++ src/core/media/apparmor/ubuntu.cpp 2015-08-18 14:11:23 +0000
4@@ -177,7 +177,9 @@
5 {
6 return Result{true, "Client can access content in /usr/share/sounds"};
7 }
8- else if (parsed_uri.scheme == "http" || parsed_uri.scheme == "rtsp")
9+ else if (parsed_uri.scheme == "http" ||
10+ parsed_uri.scheme == "https" ||
11+ parsed_uri.scheme == "rtsp")
12 {
13 return Result{true, "Client can access streaming content"};
14 }

Subscribers

People subscribed via source and target branches