@abh3 commented on this pull request.

So, we have a problem here. In general, this violates WLCG search rules. You are allowed to add additional search possibilities after all the official places you must look at are exhausted. This patch basically short-circuits that and says if it's on the URL don't look anywhere else. So, either move the code to the end of the search criteria or get WLCG dispensation. Please note the WLCG never envisioned this kind of thing so they will need to decide the policy here, I think all you have to do is work with Oliver on this.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <xrootd/xrootd/pull/1926/review/1310808335@github.com>

[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/xrootd/xrootd/pull/1926#pullrequestreview-1310808335", "url": "https://github.com/xrootd/xrootd/pull/1926#pullrequestreview-1310808335", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1