@bbockelm commented on this pull request.
In src/XrdVoms/XrdVomsHttp.cc:
> @@ -142,7 +143,9 @@ XrdHttpSecXtractor *XrdHttpGetSecXtractor(XrdHttpSecXtractorArgs) // Now return the interface object // - return (XrdHttpSecXtractor *)new XrdVomsHttp(eDest, *vomsFun); + auto base = static_cast<XrdHttpSecXtractor *>(new XrdVomsHttp(eDest, *vomsFun)); + auto wrapper = static_cast<XrdHttpSecXtractor *>(XrdVomsMapfile::Configure(eDest, base)); + return wrapper ? wrapper : base;
I did the wrapper because I thought that's what we agreed to in the phone call -- keep the code outside the existing XrdVoms class. Will be happy to put it inside the XrdVomsHttp
class itself if you're OK with that.
I am reluctant to create a new version of the plugin protocol as that would probably require me to add even more arguments to the GSI configuration, which is already much too long.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
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