<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta http-equiv="Content-Type" content="text/xhtml; charset=utf-8">
</head>
<body>
<div style="font-family:sans-serif"><div style="white-space:normal">
<p dir="auto">On 10 Dec 2016, at 15:09, Filip Roséen wrote:</p>

<blockquote style="border-left:2px solid #777; color:#777; margin:0 0 5px; padding-left:5px">
<p dir="auto">Hi Pierre,</p>

<p dir="auto">On 2016-12-10 01:08, Pierre Ynard wrote:</p>

<blockquote style="border-left:2px solid #777; color:#999; margin:0 0 5px; padding-left:5px; border-left-color:#999">
<blockquote style="border-left:2px solid #777; color:#BBB; margin:0 0 5px; padding-left:5px; border-left-color:#BBB">
<p dir="auto">Previously the implementation would unconditionally download art,<br>
these changes make sure that we respect the scope of the fetcher being<br>
used.</p>
</blockquote>

<p dir="auto">Does this mean that the Art URL returned by any lua playlist script<br>
suddenly gets ignored? There will be complaints about that, and it<br>
hardly makes sense since typical scripts parse a web page resulting from<br>
deliberate network access, and are allowed by design to make secondary<br>
network requests to fetch the media URL and other metadata.</p>
</blockquote>

<p dir="auto">It will be "ignored" if the user has specified<br>
<code style="background-color:#F7F7F7; border-radius:3px; margin:0; padding:0 0.4em" bgcolor="#F7F7F7">--no-metadata-network-access</code> and the URL for the art specified is<br>
not considered "local" (see previous discussion with <em>Remi</em> for<br>
further information on the matteR).</p>

<p dir="auto">I cannot see any other way to interpret the flag, and the intent has<br>
always been to block metadata lookup and access for the desribed set<br>
of entities (when the flag is enabled).</p>
</blockquote>

<p dir="auto">I have the same opinion as Pierre, that this would be wrong. This is not what<br>
this flag is intended to do, or if it is, it is incorrectly advertised to the user.<br>
The flag is just intended to block searching for unknown metadata, but in no way<br>
to prevent external metadata, that is not considered local, to be fetched and<br>
would probably confuse a lot of users, me included, if that would be the <br>
new behavior.</p>

<p dir="auto">If a playlist, title or lua playlist script explicitly states external metadata<br>
for an item, it should still be fetched even with <code style="background-color:#F7F7F7; border-radius:3px; margin:0; padding:0 0.4em" bgcolor="#F7F7F7">--no-metadata-network-access</code><br>
IMHO.</p>

<p dir="auto">What should <em>not</em> happen, is looking it up with external art fetchers (google or<br>
other services), if the flag is set.</p>

<blockquote style="border-left:2px solid #777; color:#777; margin:0 0 5px; padding-left:5px">
<p dir="auto">[…]</p>
</blockquote>
</div>
</div>
</body>
</html>