Print

Print

@simonmichal commented on this pull request.

In src/XrdSecgsi/XrdSecProtocolgsi.cc:

> @@ -5082,12 +5111,22 @@ int XrdSecProtocolgsi::QueryProxy(bool checkcache, XrdSutCache *cache,
                   continue;
                }
             }
-            // Parse the proxy file
-            int nci = (*ParseFile)(pi->out, po->chain);
-            if (nci < 2) {
-               DEBUG("proxy files must have at least two certificates"
-                     " (found: "<<nci<<")");
-               continue;
+            if (pi->nopxy) {
+               // Parse the cert file
+               int nci = (*ParseFile)(pi->cert, po->chain, pi->key);
+               if (nci < 1) {
+                  DEBUG("proxy files must have at least 1 certificates"

Is this log message correct, aren't we parsing here the certificate (not the proxy)?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/xrootd/xrootd/pull/1493#pullrequestreview-741628318", "url": "https://github.com/xrootd/xrootd/pull/1493#pullrequestreview-741628318", "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