Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Actually include XrdSecEntity moninfo field in trace. #1819

Closed
wants to merge 1 commit into from

Conversation

bbockelm
Copy link
Contributor

@bbockelm bbockelm commented Nov 3, 2022

This logs the monitoring information associated with the Entity (as well as fixing an annoying text alignment issue in the extended attributes). With this, the xrootd authentication trace looks as follows for GSI:

bbockelm.183358:36@hcc-briantest7 Protocol 'gsi'
bbockelm.183358:36@hcc-briantest7 Name 'bbockelm'
bbockelm.183358:36@hcc-briantest7 Host 'hcc-briantest7.unl.edu'
bbockelm.183358:36@hcc-briantest7 Vorg 'cms cms cms'
bbockelm.183358:36@hcc-briantest7 Role 'NULL NULL NULL'
bbockelm.183358:36@hcc-briantest7 Grps '/cms /cms/integration /cms/uscms'
bbockelm.183358:36@hcc-briantest7 Caps ''
bbockelm.183358:36@hcc-briantest7 Pidn 'bbockelm.183358:36@hcc-briantest7'
bbockelm.183358:36@hcc-briantest7 Mon  '/DC=ch/DC=cern/OU=Organic Units/OU=Users/CN=bbockelm/CN=659869/CN=Brian Paul Bockelman'
bbockelm.183358:36@hcc-briantest7 Crlen 0
bbockelm.183358:36@hcc-briantest7 ueid  1
bbockelm.183358:36@hcc-briantest7 uid   0
bbockelm.183358:36@hcc-briantest7 gid   0
bbockelm.183358:36@hcc-briantest7 Attr gridmap.name = '1'
bbockelm.183358:36@hcc-briantest7 Attr  = ''

bbockelm pushed a commit to opensciencegrid/Software-Redhat that referenced this pull request Nov 4, 2022
@abh3
Copy link
Member

abh3 commented Nov 18, 2022

I have taken a safer and more consistent approach by adding a gsi option that displays the subject DN when it is useful. See documentation at https://xrootd.slac.stanford.edu/doc/dev56/sec_config.htm#_Toc119617433

So, we will not be merging this. However, I will correct the nasty formatting error that you have pointed out.

@abh3 abh3 closed this Nov 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants