Well, I was trying to kill several birds with one stone. I was looking at the OSG plugin and it does a getpwnam() every single time which is rather ridiculous. That should be done once. Plus, if the plugin that you are using is this one:
https://github.com/opensciencegrid/xrootd-multiuser/blob/master/src/multiuser.cpp
it just doesn't follow the preferred plugin rules in R5 which allow you to get rid of as lot of code. Yes, will work but incredibly poorly. Frankly, instead of looking at putting in a lot of extra code on the xroot side to support this plugin, you would be far better off to just rewrite it to do something more reasonable. Once you do that then we could potentially integrate it with the main repo.

As far as serializing the SecEntity object, yes sss does that but only once per login. Here we would potentially be serializing it more than once. While cost of the fixed fields is not that high, the cost of the active fields could be quite high and not always appropriate. Frankly, you would be better off to simply fix the plugin to do something more reasonable.


You are receiving this because you are subscribed to this thread.
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/1319#issuecomment-721418251", "url": "https://github.com/xrootd/xrootd/pull/1319#issuecomment-721418251", "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