Merge lp:~dbarth/media-hub/fix-allowed-uri-scheme into lp:media-hub/stable

Proposed by David Barth
Status: Merged
Approved by: Jim Hodapp
Approved revision: 153
Merged at revision: 149
Proposed branch: lp:~dbarth/media-hub/fix-allowed-uri-scheme
Merge into: lp:media-hub/stable
Diff against target: 30 lines (+10/-2)
2 files modified
debian/changelog (+7/-1)
src/core/media/apparmor/ubuntu.cpp (+3/-1)
To merge this branch: bzr merge lp:~dbarth/media-hub/fix-allowed-uri-scheme
Reviewer Review Type Date Requested Status
Jim Hodapp (community) code Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+268343@code.launchpad.net

This proposal supersedes 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 : Posted in a previous version of this proposal

Looks good, thanks.

review: Approve (code)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
150. By David Barth

add changelog

151. By David Barth

changelog

152. By David Barth

manual change of the changelog to allow CI build to proceed

153. By David Barth

add a changelog entry for CI

Revision history for this message
Jim Hodapp (jhodapp) wrote :

Looks good

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2015-07-10 18:12:22 +0000
+++ debian/changelog 2015-08-19 07:38:21 +0000
@@ -1,4 +1,10 @@
1media-hub (3.1.0+15.10.20150710-0ubuntu1) wily; urgency=medium1media-hub (3.1.0+15.04.20150818-0ubuntu1) vivid; urgency=medium
2
3 * Add HTTPS to the list of supported URL schemes (courtesy Justin McPherson)
4
5 -- David Barth <david.barth@canonical.com> Wed, 19 Aug 2015 09:22:37 +0200
6
7media-hub (3.1.0+15.04.20150710-0ubuntu1) vivid; urgency=medium
28
3 [ CI Train Bot ]9 [ CI Train Bot ]
4 * New rebuild forced.10 * New rebuild forced.
511
=== modified file 'src/core/media/apparmor/ubuntu.cpp'
--- src/core/media/apparmor/ubuntu.cpp 2014-11-26 16:23:19 +0000
+++ src/core/media/apparmor/ubuntu.cpp 2015-08-19 07:38:21 +0000
@@ -177,7 +177,9 @@
177 {177 {
178 return Result{true, "Client can access content in /usr/share/sounds"};178 return Result{true, "Client can access content in /usr/share/sounds"};
179 }179 }
180 else if (parsed_uri.scheme == "http" || parsed_uri.scheme == "rtsp")180 else if (parsed_uri.scheme == "http" ||
181 parsed_uri.scheme == "https" ||
182 parsed_uri.scheme == "rtsp")
181 {183 {
182 return Result{true, "Client can access streaming content"};184 return Result{true, "Client can access streaming content"};
183 }185 }

Subscribers

People subscribed via source and target branches