Print

Print

Sorry for taking so long to review this. There was a problematic use of the kXR_vernnn flag. This flag is meant to describe protocol versions not client abilities. So, to speed things along I introduced a mechanism to indicate that the client supports local files and thus 'file://" urls.

Ostensibly, one should check XrdOucErrInfo::ucap for XrdOucErrInfo::uLclF and, if set, local files are supported. If XrdOucErrInfo::uUrlOK is also set then you can redirect the client to a local file. This is all in commit 684411 which is in git head.

So, please remove the changes to XProtocol.hh and XrdClXRootDTransport.cc, and change your plugin code to test these flags.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/xrootd/xrootd/pull/1022?email_source=notifications\u0026email_token=AA7NRDTBZIV2NNRFUKXLQP3QBTMVRA5CNFSM4H7PLNU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD26UPHI#issuecomment-515721117", "url": "https://github.com/xrootd/xrootd/pull/1022?email_source=notifications\u0026email_token=AA7NRDTBZIV2NNRFUKXLQP3QBTMVRA5CNFSM4H7PLNU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD26UPHI#issuecomment-515721117", "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