È sicuro per più thread impostare ServicePointManager.ServerCertificateValidationCallback?

Question

Per ignorare gli errori del certificato ssl, sto impostando ServicePointManager.ServerCertificateValidationCallback in un metodo statico prima di effettuare un HttpWebRequest. Voglio solo che questo sia fatto per le richieste interne e quindi sto reimpostando la proprietà sul suo valore predefinito nel blocco finally. Ma poiché si tratta di un'applicazione Web, ci saranno problemi quando più thread stanno modificando la proprietà?

Ecco come sto usando la proprietà

public static String GetResource() 
{ 
    try 
    {   
     ServicePointManager.ServerCertificateValidationCallback += delegate { return true; };   
    } 
    catch() 
    { 
    } 
    finally 
    { 
     ServicePointManager.ServerCertificateValidationCallback -= delegate { return false; }; 
    }  
} 

1. Sarà questo codice essere threadsafe? La documentazione su msdn dice che tutti i membri statici del tipo ServicePointManager sono thread-safe, ma volevo solo confermarli. http://msdn.microsoft.com/en-us/library/zkfa48de%28v=vs.80%29.aspx
2. Il codice nel blocco finally, è il modo corretto di reimpostarlo sul valore predefinito?

Answer 1

Se è necessario sostituire la convalida dei certificati di default, prendere in considerazione uno o più dei seguenti elementi:

1. Set ServerCertificateValidationCallback una volta e solo una volta - durante l'applicazione start up o, eventualmente, in un costruttore statico. Ciò elimina il rischio di contesa del filo.

2. Dal momento che si sta facendo la sicurezza più permissiva, limitare il comportamento di build di debug con la compilazione condizionale:

   #if DEBUG 
       ServicePointManager.ServerCertificateValidationCallback += Callback; 
   #endif 
   

3. Infine, ricordate che il delegato è una funzione ricco. Non devi semplicemente restituire true. Puoi interrogare la richiesta e decidere come gestirla.

   ServicePointManager.ServerCertificateValidationCallback += Callback; 
   
   static bool Callback(object sender, X509Certificate certificate, X509Chain chain, SslPolicyErrors sslPolicyErrors) 
   { 
       if (IsInternalRequest(sender)) 
       { 
        return true; 
       } 
       else 
       { 
        return IsExternalRequestValid(sender, certificate, chain, sslPolicyErrors); 
       } 
   }