Print

Print


@simonmichal commented on this pull request.



> @@ -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 or view it on GitHub:
https://github.com/xrootd/xrootd/pull/1493#pullrequestreview-741628318

########################################################################
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