Bump the opcfoundation group 1.5.374.117-preview (#633)

* Bump the opcfoundation group .117-preview * fix GDS build & db creation * add onDelete cascade option to EF model, fix missed CertificateStore.Open * fix ComDaClientNodeManager by removing reference to MonitoredItem Property * fix some cert store usage --------- Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Martin Regen <[email protected]> Co-authored-by: Roman Ettlinger <[email protected]>
OPCFoundation · Sep 13, 2024 · bfb3cbe · bfb3cbe
1 parent 654c562
commit bfb3cbe
Show file tree

Hide file tree

Showing 59 changed files with 130 additions and 120 deletions.
diff --git a/ComIOP/Common/Client/Da/ComDaClientNodeManager.cs b/ComIOP/Common/Client/Da/ComDaClientNodeManager.cs
@@ -825,7 +825,6 @@ protected override ServiceResult CreateMonitoredItem(ServerSystemContext context
             monitoredItem = datachangeItem;
 
             // save the monitored item.
-            MonitoredItems.Add(monitoredItemId, datachangeItem);
             monitoredNode.Add(datachangeItem);
 
             // report change.

diff --git a/ComIOP/Common/UA COM Interop Library.csproj b/ComIOP/Common/UA COM Interop Library.csproj
@@ -265,13 +265,13 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Client">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Core">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Server">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
   </ItemGroup>
   <Import Project="$(MSBuildBinPath)\Microsoft.CSharp.targets" />

diff --git a/ComIOP/Wrapper/ServerWrapper/UA COM Server Wrapper.csproj b/ComIOP/Wrapper/ServerWrapper/UA COM Server Wrapper.csproj
@@ -140,7 +140,7 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Core">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
   </ItemGroup>
   <Import Project="$(MSBuildBinPath)\Microsoft.CSharp.targets" />

diff --git a/Samples/Client.Net4/UA Sample Client.csproj b/Samples/Client.Net4/UA Sample Client.csproj
@@ -138,7 +138,7 @@
       <Version>8.0.0</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Bindings.Https">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="System.Net.Http">
       <Version>4.3.4</Version>

diff --git a/Samples/Client/Opc.Ua.SampleClient.csproj b/Samples/Client/Opc.Ua.SampleClient.csproj
@@ -158,7 +158,7 @@
       <Version>6.2.14</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Client">
-      <Version>1.5.373.121</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="System.Xml.XmlDocument">
       <Version>4.3.0</Version>

diff --git a/Samples/ClientControls.Net4/UA Client Controls.csproj b/Samples/ClientControls.Net4/UA Client Controls.csproj
@@ -1015,16 +1015,16 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Core">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.114</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Client">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.114</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Client.ComplexTypes">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.114</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Configuration">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.114</Version>
     </PackageReference>
   </ItemGroup>
   <Import Project="$(MSBuildBinPath)\Microsoft.CSharp.targets" />

diff --git a/Samples/ClientControls/Opc.Ua.Client.Controls.csproj b/Samples/ClientControls/Opc.Ua.Client.Controls.csproj
@@ -267,7 +267,7 @@
       <Version>6.2.14</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Client">
-      <Version>1.5.373.121</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="System.Xml.XmlDocument">
       <Version>4.3.0</Version>

diff --git a/Samples/Controls.Net4/UA Sample Controls.csproj b/Samples/Controls.Net4/UA Sample Controls.csproj
@@ -708,7 +708,7 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Server">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.114</Version>
     </PackageReference>
   </ItemGroup>
   <Import Project="$(MSBuildBinPath)\Microsoft.CSharp.targets" />

diff --git a/Samples/Controls/Opc.Ua.Sample.Controls.csproj b/Samples/Controls/Opc.Ua.Sample.Controls.csproj
@@ -207,7 +207,7 @@
       <Version>6.2.14</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Server">
-      <Version>1.5.373.121</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="System.Xml.XmlDocument">
       <Version>4.3.0</Version>

diff --git a/Samples/GDS/Client/Controls/ApplicationCertificateControl.cs b/Samples/GDS/Client/Controls/ApplicationCertificateControl.cs
@@ -336,7 +336,8 @@ private async void CertificateRequestTimer_Tick(object sender, EventArgs e)
                         };
 
                         // update store
-                        using (ICertificateStore store = CertificateStoreIdentifier.OpenStore(m_application.CertificateStorePath, false))
+                        var certificateStoreIdentifier = new CertificateStoreIdentifier(m_application.CertificateStorePath, false);
+                        using (ICertificateStore store = certificateStoreIdentifier.OpenStore())
                         {
                             // if we used a CSR, we already have a private key and therefore didn't request one from the GDS
                             // in this case, privateKey is null
@@ -436,7 +437,8 @@ private async void CertificateRequestTimer_Tick(object sender, EventArgs e)
                     // update trust list.
                     if (!String.IsNullOrEmpty(m_application.TrustListStorePath))
                     {
-                        using (ICertificateStore store = CertificateStoreIdentifier.OpenStore(m_application.TrustListStorePath))
+                        var certificateStoreIdentifier = new CertificateStoreIdentifier(m_application.TrustListStorePath);
+                        using (ICertificateStore store = certificateStoreIdentifier.OpenStore())
                         {
                             foreach (byte[] issuerCertificate in issuerCertificates)
                             {

diff --git a/Samples/GDS/Client/Controls/ApplicationTrustListControl.cs b/Samples/GDS/Client/Controls/ApplicationTrustListControl.cs
@@ -89,8 +89,8 @@ private void ReloadTrustListButton_Click(object sender, EventArgs e)
                     }
                     else
                     {
-                    CertificateStoreControl.Initialize(m_trustListStorePath, m_issuerListStorePath, null);
-                }
+                        CertificateStoreControl.Initialize(m_trustListStorePath, m_issuerListStorePath, null);
+                    }
                 }
                 else
                 {
@@ -120,16 +120,12 @@ private async Task DeleteExistingFromStore(string storePath)
                 return;
             }
 
-            using (DirectoryCertificateStore store = (DirectoryCertificateStore) CertificateStoreIdentifier.OpenStore(storePath))
+            var certificateStoreIdentifier = new CertificateStoreIdentifier(storePath);
+            using (var store = certificateStoreIdentifier.OpenStore())
             {
                 X509Certificate2Collection certificates = await store.Enumerate();
                 foreach (var certificate in certificates)
                 {
-                    if (store.GetPrivateKeyFilePath(certificate.Thumbprint) != null)
-                    {
-                        continue;
-                    }
-
                     List<string> fields = X509Utils.ParseDistinguishedName(certificate.Subject);
 
                     if (fields.Contains("CN=UA Local Discovery Server"))
@@ -140,6 +136,11 @@ private async Task DeleteExistingFromStore(string storePath)
 
                     if (store is DirectoryCertificateStore ds)
                     {
+                        if (ds.GetPrivateKeyFilePath(certificate.Thumbprint) != null)
+                        {
+                            continue;
+                        }
+
                         string path = Utils.GetAbsoluteFilePath(m_application.CertificatePublicKeyPath, true, false, false);
 
                         if (path != null)
@@ -205,7 +206,8 @@ private void PullFromGds(bool deleteBeforeAdd)
 
                 if (!String.IsNullOrEmpty(m_trustListStorePath))
                 {
-                    using (ICertificateStore store = CertificateStoreIdentifier.OpenStore(m_trustListStorePath))
+                    var certificateStoreIdentifier = new CertificateStoreIdentifier(m_trustListStorePath);
+                    using (ICertificateStore store = certificateStoreIdentifier.OpenStore())
                     {
                         if ((trustList.SpecifiedLists & (uint)Opc.Ua.TrustListMasks.TrustedCertificates) != 0)
                         {
@@ -233,7 +235,8 @@ private void PullFromGds(bool deleteBeforeAdd)
 
                 if (!String.IsNullOrEmpty(m_application.IssuerListStorePath))
                 {
-                    using (ICertificateStore store = CertificateStoreIdentifier.OpenStore(m_application.IssuerListStorePath))
+                    var certificateStoreIdentifier = new CertificateStoreIdentifier(m_application.IssuerListStorePath);
+                    using (ICertificateStore store = certificateStoreIdentifier.OpenStore())
                     {
                         if ((trustList.SpecifiedLists & (uint)Opc.Ua.TrustListMasks.IssuerCertificates) != 0)
                         {

diff --git a/Samples/GDS/Client/GlobalDiscoveryClient.csproj b/Samples/GDS/Client/GlobalDiscoveryClient.csproj
@@ -139,13 +139,10 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Configuration">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Gds.Client.Common">
-      <Version>1.5.374.78</Version>
-    </PackageReference>
-    <PackageReference Include="System.Net.Http">
-      <Version>4.3.4</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
   </ItemGroup>
   <ItemGroup>

diff --git a/Samples/GDS/ClientControls/Controls/TrustListControl.cs b/Samples/GDS/ClientControls/Controls/TrustListControl.cs
@@ -82,8 +82,8 @@ private enum Status
 
         private ICertificateStore CreateStore(string storePath)
         {
-            ICertificateStore store = CertificateStoreIdentifier.CreateStore(CertificateStoreIdentifier.DetermineStoreType(storePath));
-            store.Open(storePath);
+            CertificateStoreIdentifier certificateStoreIdentifier = new CertificateStoreIdentifier(storePath);
+            ICertificateStore store = certificateStoreIdentifier.OpenStore();
             return store;
         }
 

diff --git a/Samples/GDS/ClientControls/GlobalDiscoveryClientControls.csproj b/Samples/GDS/ClientControls/GlobalDiscoveryClientControls.csproj
@@ -217,10 +217,10 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Client">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Gds.Client.Common">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
   </ItemGroup>
   <ItemGroup>

diff --git a/Samples/GDS/ConsoleServer/NetCoreGlobalDiscoveryServer.csproj b/Samples/GDS/ConsoleServer/NetCoreGlobalDiscoveryServer.csproj
@@ -14,8 +14,8 @@
   <ItemGroup>
     <PackageReference Include="Microsoft.VisualStudio.Azure.Containers.Tools.Targets" Version="1.21.0" />
     <PackageReference Include="Mono.Options" Version="6.12.0.148" />
-    <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Gds.Server.Common" Version="1.5.374.78" />
-    <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Configuration" Version="1.5.374.78" />
+    <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Gds.Server.Common" Version="1.5.374.117-preview" />
+    <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Configuration" Version="1.5.374.117-preview" />
   </ItemGroup>
 
 </Project>
diff --git a/Samples/GDS/ConsoleServer/Program.cs b/Samples/GDS/ConsoleServer/Program.cs
@@ -167,20 +167,22 @@ public void Run()
                 return;
             }
 
-            ManualResetEvent quitEvent = new ManualResetEvent(false);
-            try
-            {
-                Console.CancelKeyPress += (sender, eArgs) => {
-                    quitEvent.Set();
-                    eArgs.Cancel = true;
-                };
-            }
-            catch
+            using (ManualResetEvent quitEvent = new ManualResetEvent(false))
             {
-            }
+                try
+                {
+                    Console.CancelKeyPress += (sender, eArgs) => {
+                        quitEvent.Set();
+                        eArgs.Cancel = true;
+                    };
+                }
+                catch
+                {
+                }
 
-            // wait for timeout or Ctrl-C
-            quitEvent.WaitOne();
+                // wait for timeout or Ctrl-C
+                quitEvent.WaitOne();
+            }
 
             if (server != null)
             {
@@ -318,18 +320,18 @@ void PrintSessionStatus(Session session, string reason, bool lastContact = false
         {
             lock (session.DiagnosticsLock)
             {
-                string item = String.Format("{0,9}:{1,20}:", reason, session.SessionDiagnostics.SessionName);
+                string item = Utils.Format("{0,9}:{1,20}:", reason, session.SessionDiagnostics.SessionName);
                 if (lastContact)
                 {
-                    item += String.Format("Last Event:{0:HH:mm:ss}", session.SessionDiagnostics.ClientLastContactTime.ToLocalTime());
+                    item += Utils.Format("Last Event:{0:HH:mm:ss}", session.SessionDiagnostics.ClientLastContactTime.ToLocalTime());
                 }
                 else
                 {
                     if (session.Identity != null)
                     {
-                        item += String.Format(":{0,20}", session.Identity.DisplayName);
+                        item += Utils.Format(":{0,20}", session.Identity.DisplayName);
                     }
-                    item += String.Format(":{0}", session.Id);
+                    item += Utils.Format(":{0}", session.Id);
                 }
                 Console.WriteLine(item);
             }

diff --git a/Samples/GDS/Server/DB/gdsdb.edmx b/Samples/GDS/Server/DB/gdsdb.edmx
@@ -130,7 +130,9 @@
   </Association>
   <Association Name="CertificateStoreApplication">
     <End Role="CertificateStore" Type="gdsdbModel.Store.CertificateStores" Multiplicity="*" />
-    <End Role="Application" Type="gdsdbModel.Store.Applications" Multiplicity="1" />
+    <End Role="Application" Type="gdsdbModel.Store.Applications" Multiplicity="1">
+	<OnDelete Action="Cascade" />
+    </End>
     <ReferentialConstraint>
       <Principal Role="Application">
         <PropertyRef Name="ID" />
@@ -276,7 +278,9 @@
         </Association>
         <Association Name="CertificateStoreApplication">
           <End Type="gdsdbModel.CertificateStore" Role="CertificateStore" Multiplicity="*" />
-          <End Type="gdsdbModel.Application" Role="Application" Multiplicity="1" />
+          <End Type="gdsdbModel.Application" Role="Application" Multiplicity="1" >
+            <OnDelete Action="Cascade" />
+          </End>
         </Association>
       </Schema>
     </edmx:ConceptualModels>

diff --git a/Samples/GDS/Server/DB/gdsdb.edmx.sql b/Samples/GDS/Server/DB/gdsdb.edmx.sql
@@ -32,6 +32,9 @@ GO
 IF OBJECT_ID(N'[dbo].[FK_CertificateRequests_Applications]', 'F') IS NOT NULL
     ALTER TABLE [dbo].[CertificateRequests] DROP CONSTRAINT [FK_CertificateRequests_Applications];
 GO
+IF OBJECT_ID(N'[dbo].[CertificateStoreApplication]', 'F') IS NOT NULL
+    ALTER TABLE [dbo].[CertificateStores] DROP CONSTRAINT [CertificateStoreApplication]
+GO
 
 -- --------------------------------------------------
 -- Dropping existing tables
@@ -204,7 +207,7 @@ ADD CONSTRAINT [FK_CertificateStoreApplication]
     FOREIGN KEY ([Application_ID])
     REFERENCES [dbo].[Applications]
         ([ID])
-    ON DELETE NO ACTION ON UPDATE NO ACTION;
+    ON DELETE CASCADE ON UPDATE NO ACTION;
 GO
 
 -- Creating non-clustered index for FOREIGN KEY 'FK_CertificateStoreApplication'

diff --git a/Samples/GDS/Server/GlobalDiscoveryServer.csproj b/Samples/GDS/Server/GlobalDiscoveryServer.csproj
@@ -235,10 +235,10 @@
       <Version>6.5.1</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Configuration">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Gds.Server.Common">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
   </ItemGroup>
   <PropertyGroup>

diff --git a/Samples/GDS/Server/Opc.Ua.GlobalDiscoveryServer.Config.xml b/Samples/GDS/Server/Opc.Ua.GlobalDiscoveryServer.Config.xml
@@ -136,7 +136,7 @@
           <CertificateGroupConfiguration>
             <Id>Default</Id>
             <CertificateType>RsaSha256ApplicationCertificateType</CertificateType>
-            <SubjectName>CN=IOP-2017 CA, O=OPC Foundation</SubjectName>
+            <SubjectName>CN=IOP-2024 CA, O=OPC Foundation</SubjectName>
             <BaseStorePath>%CommonApplicationData%/OPC Foundation/GDS/PKI/CA/default</BaseStorePath>
             <DefaultCertificateLifetime>12</DefaultCertificateLifetime>
             <DefaultCertificateKeySize>2048</DefaultCertificateKeySize>

diff --git a/Samples/Opc.Ua.Sample/Opc.Ua.Sample.csproj b/Samples/Opc.Ua.Sample/Opc.Ua.Sample.csproj
@@ -23,8 +23,8 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Core" Version="1.5.374.78" />
-    <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Server" Version="1.5.374.78" />
+    <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Core" Version="1.5.374.117-preview" />
+    <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Server" Version="1.5.374.117-preview" />
   </ItemGroup>
 
 </Project>
diff --git a/Samples/ReferenceClient/Reference Client.csproj b/Samples/ReferenceClient/Reference Client.csproj
@@ -127,16 +127,16 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Bindings.Https">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Client">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Configuration">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
     <PackageReference Include="OPCFoundation.NetStandard.Opc.Ua.Core">
-      <Version>1.5.374.78</Version>
+      <Version>1.5.374.117-preview</Version>
     </PackageReference>
   </ItemGroup>
   <PropertyGroup>