[vlc-devel] [PATCH 2/2] playlist/fetcher: do not ignore metadata scope when downloading art
epirat07 at gmail.com
Sun Dec 11 11:19:32 CET 2016
On 10 Dec 2016, at 15:09, Filip Roséen wrote:
> Hi Pierre,
> On 2016-12-10 01:08, Pierre Ynard wrote:
>>> Previously the implementation would unconditionally download art,
>>> these changes make sure that we respect the scope of the fetcher
>> Does this mean that the Art URL returned by any lua playlist script
>> suddenly gets ignored? There will be complaints about that, and it
>> hardly makes sense since typical scripts parse a web page resulting
>> deliberate network access, and are allowed by design to make
>> network requests to fetch the media URL and other metadata.
> It will be "ignored" if the user has specified
> `--no-metadata-network-access` and the URL for the art specified is
> not considered "local" (see previous discussion with *Remi* for
> further information on the matteR).
> I cannot see any other way to interpret the flag, and the intent has
> always been to block metadata lookup and access for the desribed set
> of entities (when the flag is enabled).
I have the same opinion as Pierre, that this would be wrong. This is not
this flag is intended to do, or if it is, it is incorrectly advertised
to the user.
The flag is just intended to block searching for unknown metadata, but
in no way
to prevent external metadata, that is not considered local, to be
would probably confuse a lot of users, me included, if that would be the
If a playlist, title or lua playlist script explicitly states external
for an item, it should still be fetched even with
What should _not_ happen, is looking it up with external art fetchers
other services), if the flag is set.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel