From 809b0eb1f52289894554def6bf078a87c9d64cf3 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sun, 29 Dec 2013 14:40:24 +0100 Subject: command: "lsinfo" and "readcomments" allowed for remote files --- NEWS | 1 + doc/protocol.xml | 8 ++++++++ src/Song.hxx | 1 + src/SongUpdate.cxx | 14 ++++++++++++++ src/command/FileCommands.cxx | 24 ++++++++++++++++++++++++ src/command/OtherCommands.cxx | 20 ++++++++++++++++++++ 6 files changed, 68 insertions(+) diff --git a/NEWS b/NEWS index 125be935f..4afd66555 100644 --- a/NEWS +++ b/NEWS @@ -1,6 +1,7 @@ ver 0.19 (not yet released) * protocol - new commands "addtagid", "cleartagid" + - "lsinfo" and "readcomments" allowed for remote files * archive - read tags from songs in an archive * input diff --git a/doc/protocol.xml b/doc/protocol.xml index a0211bffd..625fef874 100644 --- a/doc/protocol.xml +++ b/doc/protocol.xml @@ -1617,6 +1617,10 @@ OK the list of stored playlists. This behavior is deprecated; use "listplaylists" instead. + + This command may be used to list metadata of remote + files (e.g. URI beginning with "http://" or "smb://"). + Clients that are connected via UNIX domain socket may use this command to read the tags of an arbitrary local @@ -1638,6 +1642,10 @@ OK to the music directory or a URL in the form "file:///foo/bar.ogg". + + This command may be used to list metadata of remote + files (e.g. URI beginning with "http://" or "smb://"). + The response consists of lines in the form "KEY: VALUE". Comments with suspicious characters (e.g. newlines) are diff --git a/src/Song.hxx b/src/Song.hxx index 21e158560..b9f529702 100644 --- a/src/Song.hxx +++ b/src/Song.hxx @@ -130,6 +130,7 @@ struct Song { bool UpdateFile(); bool UpdateFileInArchive(); + bool UpdateStream(); /** * Returns the URI of the song in UTF-8 encoding, including its diff --git a/src/SongUpdate.cxx b/src/SongUpdate.cxx index fd4aa6751..aef484a6f 100644 --- a/src/SongUpdate.cxx +++ b/src/SongUpdate.cxx @@ -132,3 +132,17 @@ Song::UpdateFileInArchive() tag = tag_builder.Commit(); return true; } + +bool +Song::UpdateStream() +{ + assert(!IsFile()); + + TagBuilder tag_builder; + if (!tag_stream_scan(uri, full_tag_handler, &tag_builder)) + return false; + + delete tag; + tag = tag_builder.Commit(); + return true; +} diff --git a/src/command/FileCommands.cxx b/src/command/FileCommands.cxx index 48037bc72..9c7118bcb 100644 --- a/src/command/FileCommands.cxx +++ b/src/command/FileCommands.cxx @@ -25,13 +25,16 @@ #include "ClientFile.hxx" #include "Client.hxx" #include "util/CharUtil.hxx" +#include "util/UriUtil.hxx" #include "util/Error.hxx" #include "tag/TagHandler.hxx" #include "tag/ApeTag.hxx" #include "tag/TagId3.hxx" +#include "TagStream.hxx" #include "TagFile.hxx" #include "Mapper.hxx" #include "fs/AllocatedPath.hxx" +#include "ls.hxx" #include @@ -80,6 +83,25 @@ static constexpr tag_handler print_comment_handler = { print_pair, }; +static CommandResult +read_stream_comments(Client &client, const char *uri) +{ + if (!uri_supported_scheme(uri)) { + command_error(client, ACK_ERROR_NO_EXIST, + "unsupported URI scheme"); + return CommandResult::ERROR; + } + + if (!tag_stream_scan(uri, print_comment_handler, &client)) { + command_error(client, ACK_ERROR_NO_EXIST, + "Failed to load file"); + return CommandResult::ERROR; + } + + return CommandResult::OK; + +} + CommandResult handle_read_comments(Client &client, gcc_unused int argc, char *argv[]) { @@ -102,6 +124,8 @@ handle_read_comments(Client &client, gcc_unused int argc, char *argv[]) Error error; if (!client_allow_file(client, path_fs, error)) return print_error(client, error); + } else if (uri_has_scheme(uri)) { + return read_stream_comments(client, uri); } else if (*uri != '/') { path_fs = map_uri_fs(uri); if (path_fs.IsNull()) { diff --git a/src/command/OtherCommands.cxx b/src/command/OtherCommands.cxx index 0c1de8792..e1cbdb618 100644 --- a/src/command/OtherCommands.cxx +++ b/src/command/OtherCommands.cxx @@ -136,6 +136,26 @@ handle_lsinfo(Client &client, int argc, char *argv[]) return CommandResult::OK; } + if (uri_has_scheme(uri)) { + if (!uri_supported_scheme(uri)) { + command_error(client, ACK_ERROR_NO_EXIST, + "unsupported URI scheme"); + return CommandResult::ERROR; + } + + Song *song = Song::NewRemote(uri); + if (!song->UpdateStream()) { + song->Free(); + command_error(client, ACK_ERROR_NO_EXIST, + "No such file"); + return CommandResult::ERROR; + } + + song_print_info(client, *song); + song->Free(); + return CommandResult::OK; + } + CommandResult result = handle_lsinfo2(client, argc, argv); if (result != CommandResult::OK) return result; -- cgit v1.2.3