Fix critical delete bug: don't mass-delete on EntryID change

Two fixes for delete propagation in Phase 1:

1. When Outlook contact not found by EntryID, try to re-match
   by name/email/phone before assuming it was deleted. Outlook
   can change EntryIDs on restart or profile changes, causing
   the sync to think ALL contacts were deleted.

2. When Starface contact not found in current list, DON'T delete
   from Outlook. The contact may belong to a different address
   book. Just drop the mapping and let Phase 2/3 re-link it.

These changes make delete propagation much safer and prevent
accidental mass-deletion of contacts.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

README.md

@@ -42,6 +42,32 @@ Windows-Anwendung zur bidirektionalen Synchronisation von Kontakten zwischen Mic
 5. Starface-Adressbuch und Outlook-Kontaktordner waehlen
 6. Speichern und "Jetzt synchronisieren"
 
+### Outlook-Sicherheitsabfrage unterdruecken
+
+Beim Zugriff auf Outlook-Kontakte zeigt Outlook standardmaessig einen
+Sicherheitsdialog ("Ein Programm versucht auf Ihre E-Mail-Adressinformationen
+zuzugreifen"). Dieser kann in den Einstellungen der App deaktiviert werden:
+
+1. In der App auf "Einstellungen" klicken
+2. "Outlook-Sicherheitsabfrage automatisch erlauben" aktivieren
+3. Speichern
+
+**Auf Domaenen-PCs / Terminal Servern:**
+
+Die Outlook-Sicherheitseinstellungen werden dort per Gruppenrichtlinie (GPO)
+gesteuert und sind im Trust Center ausgegraut. In diesem Fall muss die App
+**einmalig als Administrator** gestartet werden, damit die Registry-Keys
+unter HKLM geschrieben werden koennen:
+
+1. Rechtsklick auf die App -> "Als Administrator ausfuehren"
+2. Einstellungen -> "Outlook-Sicherheitsabfrage automatisch erlauben" aktivieren
+3. Speichern und App schliessen
+4. Outlook neu starten
+5. App kann danach wieder normal (ohne Admin) gestartet werden
+
+Die Einstellung bleibt dauerhaft bestehen und gilt fuer alle Benutzer
+auf dem Rechner.
+
 ### Deinstallation
 
 Ueber Windows Einstellungen -> Apps oder die Systemsteuerung.

installer/setup.iss

@@ -2,7 +2,7 @@
 ; Erfordert Inno Setup 6.x (https://jrsoftware.org/isinfo.php)
 
 #define MyAppName "Starface Outlook Sync"
-#define MyAppVersion "0.0.0.21"
+#define MyAppVersion "0.0.0.23"
 #define MyAppPublisher "HackerSoft - Hacker-Net Telekommunikation"
 #define MyAppURL "https://www.hacker-net.de"
 #define MyAppExeName "StarfaceOutlookSync.exe"

src/StarfaceOutlookSync/Models/UserSettings.cs

@@ -41,42 +41,80 @@ namespace StarfaceOutlookSync.Models
 
         public void ApplyOutlookSecuritySetting()
         {
-            // Alle Office-Versionen abdecken (16.0 = 2016/2019/2021/2024/365, 15.0 = 2013)
             var versions = new[] { "16.0", "15.0" };
 
+            var securityValues = new (string name, int value)[]
+            {
+                ("ObjectModelGuard", 2),
+                ("PromptOOMAddressBookAccess", 2),
+                ("PromptOOMAddressInformationAccess", 2),
+                ("PromptOOMSend", 2),
+                ("PromptOOMSaveAs", 2),
+                ("PromptOOMFormulaAccess", 2),
+                ("PromptOOMCustomAction", 2),
+                ("PromptSimpleMAPISend", 2),
+                ("PromptSimpleMAPINameResolve", 2),
+                ("PromptSimpleMAPIOpenMessage", 2),
+                ("AdminSecurityMode", 3),
+            };
+
+            // In alle moeglichen Pfade schreiben (HKCU + HKLM, Policies + direkt)
+            var roots = new[] { Registry.CurrentUser, Registry.LocalMachine };
+            var prefixes = new[]
+            {
+                @"Software\Policies\Microsoft\Office",
+                @"Software\Microsoft\Office"
+            };
+
             foreach (var ver in versions)
             {
-                try
+                foreach (var root in roots)
                 {
-                    var regPath = $@"Software\Policies\Microsoft\Office\{ver}\Outlook\Security";
-
-                    if (AutoAcceptOutlookPrompt)
-                    {
-                        var key = Registry.CurrentUser.CreateSubKey(regPath);
-                        key.SetValue("ObjectModelGuard", 2, RegistryValueKind.DWord);
-                        key.SetValue("PromptOOMAddressBookAccess", 2, RegistryValueKind.DWord);
-                        key.SetValue("PromptOOMAddressInformationAccess", 2, RegistryValueKind.DWord);
-                        key.SetValue("PromptOOMSend", 2, RegistryValueKind.DWord);
-                        key.SetValue("PromptOOMSaveAs", 2, RegistryValueKind.DWord);
-                        key.SetValue("PromptOOMFormulaAccess", 2, RegistryValueKind.DWord);
-                        key.SetValue("PromptOOMCustomAction", 2, RegistryValueKind.DWord);
-                        key.SetValue("PromptSimpleMAPISend", 2, RegistryValueKind.DWord);
-                        key.SetValue("PromptSimpleMAPINameResolve", 2, RegistryValueKind.DWord);
-                        key.SetValue("PromptSimpleMAPIOpenMessage", 2, RegistryValueKind.DWord);
-                        key.SetValue("AdminSecurityMode", 3, RegistryValueKind.DWord);
-                        key.Close();
-                    }
-                    else
+                    foreach (var prefix in prefixes)
                     {
+                        var regPath = $@"{prefix}\{ver}\Outlook\Security";
                         try
                         {
-                            Registry.CurrentUser.DeleteSubKey(regPath, false);
+                            if (AutoAcceptOutlookPrompt)
+                            {
+                                var key = root.CreateSubKey(regPath);
+                                if (key != null)
+                                {
+                                    foreach (var (name, value) in securityValues)
+                                        key.SetValue(name, value, RegistryValueKind.DWord);
+                                    key.Close();
+                                }
+                            }
+                            else
+                            {
+                                try { root.DeleteSubKey(regPath, false); } catch { }
+                            }
                         }
-                        catch { }
+                        catch { } // Kein Fehler wenn Rechte fehlen - naechsten Pfad versuchen
                     }
                 }
-                catch { }
             }
         }
+
+        /// <summary>
+        /// Prueft ob die Outlook-Sicherheitseinstellung per GPO blockiert wird.
+        /// </summary>
+        public static bool IsOutlookSecurityLockedByPolicy()
+        {
+            try
+            {
+                // Wenn HKLM Policies gesetzt sind und wir dort nicht schreiben koennen
+                var key = Registry.LocalMachine.OpenSubKey(
+                    @"Software\Policies\Microsoft\Office\16.0\Outlook\Security", false);
+                if (key != null)
+                {
+                    var val = key.GetValue("AdminSecurityMode");
+                    key.Close();
+                    if (val != null) return true;
+                }
+            }
+            catch { }
+            return false;
+        }
     }
 }

src/StarfaceOutlookSync/Services/SyncEngine.cs

@@ -183,7 +183,21 @@ namespace StarfaceOutlookSync.Services
 
                         if (oc == null && sc != null)
                         {
-                            // In Outlook geloescht -> in Starface auch loeschen
+                            // Outlook-Kontakt nicht gefunden.
+                            // Erst pruefen ob er vielleicht nur eine neue EntryID hat
+                            var reMatch = FindMatch(sc, outlookContacts.Where(c =>
+                                !processedOutlookIds.Contains(c.OutlookEntryId)).ToList());
+                            if (reMatch != null)
+                            {
+                                // Kontakt existiert noch in Outlook, nur EntryID geaendert
+                                Log($"  EntryID geaendert, verknuepfe neu: {sc.DisplayName}");
+                                mapping.OutlookEntryId = reMatch.OutlookEntryId;
+                                processedOutlookIds.Add(reMatch.OutlookEntryId);
+                                newMappings.Add(mapping);
+                                continue;
+                            }
+
+                            // Wirklich geloescht -> in Starface auch loeschen
                             if (profile.SyncDirection == SyncDirection.Both || profile.SyncDirection == SyncDirection.OutlookToStarface)
                             {
                                 if (await starface.DeleteContactAsync(mapping.StarfaceId))
@@ -194,7 +208,6 @@ namespace StarfaceOutlookSync.Services
                             }
                             else
                             {
-                                // Richtung erlaubt kein Loeschen -> Mapping behalten
                                 newMappings.Add(mapping);
                             }
                             continue;
@@ -202,20 +215,12 @@ namespace StarfaceOutlookSync.Services
 
                         if (oc != null && sc == null)
                         {
-                            // In Starface geloescht -> in Outlook auch loeschen
-                            if (profile.SyncDirection == SyncDirection.Both || profile.SyncDirection == SyncDirection.StarfaceToOutlook)
-                            {
-                                if (_outlookService.DeleteContact(mapping.OutlookEntryId))
-                                {
-                                    result.Updated++;
-                                    Log($"  Geloescht (SF->OL): {oc.DisplayName}");
-                                }
-                            }
-                            else
-                            {
-                                // Richtung erlaubt kein Loeschen -> Mapping behalten
-                                newMappings.Add(mapping);
-                            }
+                            // Starface-Kontakt nicht gefunden.
+                            // Kann passieren wenn der Kontakt einem anderen Adressbuch gehoert.
+                            // NICHT loeschen, nur Mapping entfernen - wird in Phase 2/3 neu verknuepft
+                            Log($"  Starface-Kontakt nicht in Liste (anderes Adressbuch?): {oc.DisplayName}");
+                            // Mapping verwerfen, Outlook-Kontakt als unverarbeitet belassen
+                            // damit er in Phase 2 neu zugeordnet oder erstellt werden kann
                             continue;
                         }
 

src/StarfaceOutlookSync/StarfaceOutlookSync.csproj

@@ -7,9 +7,9 @@
     <AssemblyTitle>Starface Outlook Sync</AssemblyTitle>
     <Company>HackerSoft - Hacker-Net Telekommunikation</Company>
     <Product>Starface Outlook Sync</Product>
-    <Version>0.0.0.21</Version>
-    <AssemblyVersion>0.0.0.21</AssemblyVersion>
-    <FileVersion>0.0.0.21</FileVersion>
+    <Version>0.0.0.23</Version>
+    <AssemblyVersion>0.0.0.23</AssemblyVersion>
+    <FileVersion>0.0.0.23</FileVersion>
     <Description>Synchronisiert Outlook-Kontakte mit Starface Telefonanlage</Description>
     <Copyright>Stefan Hacker - HackerSoft</Copyright>
     <RuntimeIdentifier>win-x64</RuntimeIdentifier>

src/StarfaceOutlookSync/UI/AboutForm.cs

@@ -27,7 +27,7 @@ namespace StarfaceOutlookSync.UI
 
             var lblVersion = new Label
             {
-                Text = "Version 0.0.0.21",
+                Text = "Version 0.0.0.23",
                 Left = 0, Top = 56, Width = 340, Height = 20,
                 TextAlign = ContentAlignment.MiddleCenter,
                 ForeColor = Color.Gray

src/StarfaceOutlookSync/UI/SettingsForm.cs

@@ -47,11 +47,15 @@ namespace StarfaceOutlookSync.UI
                 Checked = _settings.AutoAcceptOutlookPrompt
             };
 
+            var hintText = "Hinweis: Outlook muss nach Aenderung neu gestartet werden.";
+            if (UserSettings.IsOutlookSecurityLockedByPolicy())
+                hintText += "\nAuf Domaenen-PCs: App einmalig als Admin starten!";
+
             var lblHint = new Label
             {
-                Text = "Hinweis: Outlook muss nach Aenderung dieser Option\nneu gestartet werden.",
-                Left = 38, Top = 102, Width = 300, Height = 32,
-                ForeColor = Color.Gray,
+                Text = hintText,
+                Left = 38, Top = 102, Width = 310, Height = 36,
+                ForeColor = UserSettings.IsOutlookSecurityLockedByPolicy() ? Color.OrangeRed : Color.Gray,
                 Font = new Font("Segoe UI", 8)
             };