[rygel/wip/cablelabs-integration: 12/18] wip: Serialize resources
- From: Jens Georg <jensgeorg src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [rygel/wip/cablelabs-integration: 12/18] wip: Serialize resources
- Date: Thu, 20 Nov 2014 20:38:44 +0000 (UTC)
commit a43468f38c6586e98b5213e5790829bf4fafb08b
Author: Jens Georg <mail jensge org>
Date: Sun Nov 16 17:52:01 2014 +0100
wip: Serialize resources
Signed-off-by: Jens Georg <mail jensge org>
src/librygel-server/rygel-http-get-handler.vala | 1 +
src/librygel-server/rygel-http-item-uri.vala | 24 ++++++++-
src/librygel-server/rygel-http-server.vala | 20 ++++++-
src/librygel-server/rygel-media-file-item.vala | 23 ---------
src/librygel-server/rygel-media-object.vala | 58 ++++++++++++++++++++++
src/librygel-server/rygel-media-resource.vala | 2 +-
src/librygel-server/rygel-music-item.vala | 1 +
src/librygel-server/rygel-transcode-manager.vala | 3 +-
src/librygel-server/rygel-transcoder.vala | 1 +
src/librygel-server/rygel-video-item.vala | 1 +
src/librygel-server/rygel-visual-item.vala | 1 +
11 files changed, 105 insertions(+), 30 deletions(-)
---
diff --git a/src/librygel-server/rygel-http-get-handler.vala b/src/librygel-server/rygel-http-get-handler.vala
index 48638ec..496d629 100644
--- a/src/librygel-server/rygel-http-get-handler.vala
+++ b/src/librygel-server/rygel-http-get-handler.vala
@@ -66,6 +66,7 @@ internal abstract class Rygel.HTTPGetHandler: GLib.Object {
// headers such as Samsung's subtitle stuff.
if (request.hack != null) {
request.hack.modify_headers (request);
+ null,
}
request.msg.response_headers.append ("Connection", "close");
diff --git a/src/librygel-server/rygel-http-item-uri.vala b/src/librygel-server/rygel-http-item-uri.vala
index 914288b..99e81cb 100644
--- a/src/librygel-server/rygel-http-item-uri.vala
+++ b/src/librygel-server/rygel-http-item-uri.vala
@@ -31,6 +31,7 @@ internal class Rygel.HTTPItemURI : Object {
public int subtitle_index { get; set; default = -1; }
public string? transcode_target { get; set; default = null; }
public string? playlist_format { get; set; default = null; }
+ public string? resource_name { get; set; default = null; }
public unowned HTTPServer http_server { get; set; }
private string real_extension;
@@ -53,17 +54,29 @@ internal class Rygel.HTTPItemURI : Object {
int thumbnail_index = -1,
int subtitle_index = -1,
string? transcode_target = null,
- string? playlist_format = null) {
+ string? playlist_format = null,
+ string? resource_name = null) {
this.item_id = object.id;
this.thumbnail_index = thumbnail_index;
this.subtitle_index = subtitle_index;
this.transcode_target = transcode_target;
this.http_server = http_server;
this.playlist_format = playlist_format;
+ this.resource_name = resource_name;
this.extension = "";
- if (!(object is MediaFileItem)) {
+ if (this.resource_name != null) {
+ var resource = object.get_resource_by_name
+ (this.resource_name);
+ if (resource != null) {
+ this.extension = resource.extension;
+ }
+
return;
+ } else {
+ if (!(object is MediaFileItem)) {
+ return;
+ }
}
var item = object as MediaFileItem;
@@ -187,6 +200,10 @@ internal class Rygel.HTTPItemURI : Object {
this.playlist_format = Soup.URI.decode (parts[i + 1]);
break;
+ case "res":
+ this.resource_name = Soup.URI.decode (parts[i + 1]);
+
+ break;
default:
break;
}
@@ -216,6 +233,9 @@ internal class Rygel.HTTPItemURI : Object {
} else if (this.playlist_format != null) {
path += "/pl/" + Uri.escape_string
(this.playlist_format, "", true);
+ } else if (this.resource_name != null) {
+ path += "/res/" + Uri.escape_string
+ (this.resource_name, "", true);
}
path += this.extension;
diff --git a/src/librygel-server/rygel-http-server.vala b/src/librygel-server/rygel-http-server.vala
index 2559d91..8c987f9 100644
--- a/src/librygel-server/rygel-http-server.vala
+++ b/src/librygel-server/rygel-http-server.vala
@@ -58,6 +58,18 @@ public class Rygel.HTTPServer : Rygel.TranscodeManager, Rygel.StateMachine {
}
}
+ /**
+ * Set or unset options the server supports/doesn't support
+ *
+ * Resources should be setup assuming server supports all optional delivery modes
+ */
+ public void set_resource_delivery_options (MediaResource res) {
+ res.protocol = this.get_protocol ();
+ // Set this just to be safe
+ res.dlna_flags |= DLNAFlags.DLNA_V15;
+ // This server supports all DLNA delivery modes - so leave those flags alone
+ }
+
internal void add_proxy_resource (DIDLLiteItem didl_item,
MediaFileItem item)
throws Error {
@@ -65,7 +77,7 @@ public class Rygel.HTTPServer : Rygel.TranscodeManager, Rygel.StateMachine {
return;
}
- var uri = this.create_uri_for_object (item, -1, -1, null, null);
+ var uri = this.create_uri_for_object (item, -1, -1, null, null, null);
item.add_resource (didl_item, uri, this.get_protocol (), uri);
}
@@ -101,13 +113,15 @@ public class Rygel.HTTPServer : Rygel.TranscodeManager, Rygel.StateMachine {
int thumbnail_index,
int subtitle_index,
string? transcode_target,
- string? playlist_target) {
+ string? playlist_target,
+ string? resource_name) {
var uri = new HTTPItemURI (object,
this,
thumbnail_index,
subtitle_index,
transcode_target,
- playlist_target);
+ playlist_target,
+ resource_name);
return uri.to_string ();
}
diff --git a/src/librygel-server/rygel-media-file-item.vala b/src/librygel-server/rygel-media-file-item.vala
index 77d33ec..a410f04 100644
--- a/src/librygel-server/rygel-media-file-item.vala
+++ b/src/librygel-server/rygel-media-file-item.vala
@@ -255,29 +255,6 @@ public abstract class Rygel.MediaFileItem : MediaItem {
return protocol_info;
}
- internal string get_protocol_for_uri (string uri) throws Error {
- var scheme = Uri.parse_scheme (uri);
- if (scheme == null) {
- throw new MediaItemError.BAD_URI (_("Bad URI: %s"), uri);
- }
-
- if (scheme == "http") {
- return "http-get";
- } else if (scheme == "file") {
- return "internal";
- } else if (scheme == "rtsp") {
- // FIXME: Assuming that RTSP is always accompanied with RTP over UDP
- return "rtsp-rtp-udp";
- } else {
- // Assume the protocol to be the scheme of the URI
- warning (_("Failed to probe protocol for URI %s. Assuming '%s'"),
- uri,
- scheme);
-
- return scheme;
- }
- }
-
protected virtual void add_resources (DIDLLiteItem didl_item,
bool allow_internal)
throws Error {
diff --git a/src/librygel-server/rygel-media-object.vala b/src/librygel-server/rygel-media-object.vala
index 2095a7c..2917c8a 100644
--- a/src/librygel-server/rygel-media-object.vala
+++ b/src/librygel-server/rygel-media-object.vala
@@ -224,6 +224,16 @@ public abstract class Rygel.MediaObject : GLib.Object {
return media_resources;
}
+ public MediaResource? get_resource_by_name (string resource_name) {
+ foreach (var resource in this.media_resources) {
+ if (resource.get_name () == resource_name) {
+ return resource;
+ }
+ }
+
+ return null;
+ }
+
public abstract DIDLLiteObject? serialize (Serializer serializer,
HTTPServer http_server)
throws Error;
@@ -242,6 +252,31 @@ public abstract class Rygel.MediaObject : GLib.Object {
public void serialize_resource_list (DIDLLiteObject didl_object,
HTTPServer http_server)
throws Error {
+ foreach (var res in this.get_resource_list ()) {
+ if (res.uri == null || res.uri == "") {
+ res.uri = http_server.create_uri_for_object (this,
+ -1,
+ -1,
+ null,
+ null,
+ res.get_name ());
+ var didl_resource = didl_object.add_resource ();
+ http_server.set_resource_delivery_options (res);
+ res.serialize (didl_resource);
+ res.uri = null;
+ } else {
+ try {
+ var protocol = this.get_protocol_for_uri (res.uri);
+ if (protocol != "internal") {
+ // Exclude internal resources when request is non-local
+ var didl_resource = didl_object.add_resource ();
+ res.serialize (didl_resource);
+ }
+ } catch (Error e) {
+ warning (_("Could not determine protocol for %s"), res.uri);
+ }
+ }
+ }
}
@@ -432,4 +467,27 @@ public abstract class Rygel.MediaObject : GLib.Object {
return "";
}
+
+ internal string get_protocol_for_uri (string uri) throws Error {
+ var scheme = Uri.parse_scheme (uri);
+ if (scheme == null) {
+ throw new MediaItemError.BAD_URI (_("Bad URI: %s"), uri);
+ }
+
+ if (scheme == "http") {
+ return "http-get";
+ } else if (scheme == "file") {
+ return "internal";
+ } else if (scheme == "rtsp") {
+ // FIXME: Assuming that RTSP is always accompanied with RTP over UDP
+ return "rtsp-rtp-udp";
+ } else {
+ // Assume the protocol to be the scheme of the URI
+ warning (_("Failed to probe protocol for URI %s. Assuming '%s'"),
+ uri,
+ scheme);
+
+ return scheme;
+ }
+ }
}
diff --git a/src/librygel-server/rygel-media-resource.vala b/src/librygel-server/rygel-media-resource.vala
index 8bbf226..0520734 100644
--- a/src/librygel-server/rygel-media-resource.vala
+++ b/src/librygel-server/rygel-media-resource.vala
@@ -170,7 +170,7 @@ public class Rygel.MediaResource : GLib.Object {
didl_resource.height = this.height;
didl_resource.audio_channels = this.audio_channels;
didl_resource.sample_freq = this.sample_freq;
- didl_resource.protocol_info = get_protocol_info ();
+ didl_resource.protocol_info = this.get_protocol_info ();
return didl_resource;
}
diff --git a/src/librygel-server/rygel-music-item.vala b/src/librygel-server/rygel-music-item.vala
index 9c48f5d..43678f8 100644
--- a/src/librygel-server/rygel-music-item.vala
+++ b/src/librygel-server/rygel-music-item.vala
@@ -135,6 +135,7 @@ public class Rygel.MusicItem : AudioItem {
0,
-1,
null,
+ null,
null);
}
}
diff --git a/src/librygel-server/rygel-transcode-manager.vala
b/src/librygel-server/rygel-transcode-manager.vala
index ddc45ff..916ef04 100644
--- a/src/librygel-server/rygel-transcode-manager.vala
+++ b/src/librygel-server/rygel-transcode-manager.vala
@@ -45,7 +45,8 @@ public abstract class Rygel.TranscodeManager : GLib.Object {
int thumbnail_index,
int subtitle_index,
string? transcode_target,
- string? playlist_target);
+ string? playlist_target,
+ string? resource_name);
public void add_resources (DIDLLiteItem didl_item, MediaFileItem item)
throws Error {
diff --git a/src/librygel-server/rygel-transcoder.vala b/src/librygel-server/rygel-transcoder.vala
index fb48fd3..efa6152 100644
--- a/src/librygel-server/rygel-transcoder.vala
+++ b/src/librygel-server/rygel-transcoder.vala
@@ -83,6 +83,7 @@ public abstract class Rygel.Transcoder : GLib.Object {
-1,
-1,
this.dlna_profile,
+ null,
null);
var res = item.add_resource (didl_item, uri, protocol);
res.size = -1;
diff --git a/src/librygel-server/rygel-video-item.vala b/src/librygel-server/rygel-video-item.vala
index c9d3a16..514bbd1 100644
--- a/src/librygel-server/rygel-video-item.vala
+++ b/src/librygel-server/rygel-video-item.vala
@@ -195,6 +195,7 @@ public class Rygel.VideoItem : AudioItem, VisualItem {
-1,
index,
null,
+ null,
null);
subtitle.add_didl_node (didl_item);
diff --git a/src/librygel-server/rygel-visual-item.vala b/src/librygel-server/rygel-visual-item.vala
index 2ab8897..254d61d 100644
--- a/src/librygel-server/rygel-visual-item.vala
+++ b/src/librygel-server/rygel-visual-item.vala
@@ -101,6 +101,7 @@ public interface Rygel.VisualItem : MediaFileItem {
index,
-1,
null,
+ null,
null);
thumbnail.add_resource (didl_item, server.get_protocol ());
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
