Print

Print


This sounds very promising!

(One aside on the manual text for this - I was very confused by this sentence in the manual:

++        The specified plug-in should stack on top of the existing plug-in or default. Once specified, it cannot be overridden by a subsequent directive,

I read this as saying "this can only be used once", which somewhat contradicts what you wrote above. A clarification in the manual would be appreciated.)

The second unpleasant piece in this code is the fact that, within the HTTP handler, we need to have access to the XrdAccAuthorize object used by the OFS to generate the appropriate permissions for the Macaroon.

Is there such a way to do that? This is currently accomplished by directly constructing the object and via a reimplementation of the ofs.authlib logic. There's not much gain if we significantly clean up this usage only to add more complexity elsewhere. What you point out only helps the case of making an authorization object for the ofs and not recreating it in the plugin.

Thanks!


You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

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