2.19.1 disable network traffic

src/DynamoApplications/StartupUtils.cs

@@ -398,20 +398,21 @@ private static DynamoModel StartDynamoWithDefaultConfig(bool CLImode,
             HostAnalyticsInfo info = new HostAnalyticsInfo(),
             bool isServiceMode = false)
         {
+
             var config = new DynamoModel.DefaultStartConfiguration
             {
                 GeometryFactoryPath = geometryFactoryPath,
                 ProcessMode = CLImode ? TaskProcessMode.Synchronous : TaskProcessMode.Asynchronous,
                 HostAnalyticsInfo = info,
                 CLIMode = CLImode,
-                AuthProvider = CLImode ? null : new Core.IDSDKManager(),
+                //TODO we currently use this like a no network comms flags - work on introducing a new flag or renaming this flag.
+                AuthProvider = CLImode || Dynamo.Logging.Analytics.DisableAnalytics ? null : new Core.IDSDKManager(),
                 UpdateManager = CLImode ? null : OSHelper.IsWindows() ? InitializeUpdateManager() : null,
                 StartInTestMode = CLImode,
                 PathResolver = CreatePathResolver(CLImode, preloaderLocation, userDataFolder, commonDataFolder),
                 IsServiceMode = isServiceMode,
                 Preferences = PreferenceSettings.Instance
             };
-
             var model = DynamoModel.Start(config);
             return model;
         }

src/DynamoCore/Logging/AnalyticsService.cs

@@ -84,6 +84,12 @@ internal static bool IsADPOptedIn
 
         internal static bool IsADPAvailable()
         {
+            if (Analytics.DisableAnalytics ||
+                adpAnalyticsUI == null)
+            {
+                return false;
+            }
+
             return adpAnalyticsUI.IsProviderAvailable();
         }
 

src/DynamoCore/Models/DynamoModel.cs

@@ -761,10 +761,6 @@ protected DynamoModel(IStartConfiguration config)
                     }
                     catch (Exception e) { Logger.LogError($"could not start feature flags manager {e}"); };
                 });
-
-                //TODO just a test of feature flag event, safe to remove at any time.
-                DynamoUtilities.DynamoFeatureFlagsManager.FlagsRetrieved += CheckFeatureFlagTest;
-
             }
 
             // TBD: Do we need settings migrator for service mode? If we config the docker correctly, this could be skipped I think
@@ -1107,31 +1103,6 @@ private IPreferences CreateOrLoadPreferences(IPreferences preferences)
             return new PreferenceSettings();
         }
 
-        private void CheckFeatureFlagTest()
-        {
-            if (!DynamoModel.IsTestMode)
-            {
-                if (DynamoModel.FeatureFlags.CheckFeatureFlag<bool>("EasterEggIcon1", false))
-                {
-                    this.Logger.Log("EasterEggIcon1 is true FROM MODEL");
-
-                }
-                else
-                {
-                    this.Logger.Log("EasterEggIcon1 is false FROM MODEL");
-                }
-
-                if (DynamoModel.FeatureFlags.CheckFeatureFlag<string>("EasterEggMessage1", "NA") is var s && s != "NA")
-                {
-                    this.Logger.Log("EasterEggMessage1 is enabled FROM MODEL");
-                }
-                else
-                {
-                    this.Logger.Log("EasterEggMessage1 is disabled FROM MODEL");
-                }
-            }
-        }
-
         private void SetDefaultPythonTemplate()
         {
             // First check if the default python template is overridden by a PythonTemplate.py file in AppData
@@ -1475,7 +1446,6 @@ public void Dispose()
             {
                 FeatureFlags.MessageLogged -= LogMessageWrapper;
             }
-            DynamoUtilities.DynamoFeatureFlagsManager.FlagsRetrieved -= CheckFeatureFlagTest;
         }
 
         private void InitializeCustomNodeManager()

src/DynamoCore/Scheduler/UpdateRenderPackageAsyncTask.cs

@@ -230,7 +230,7 @@ private void GetTessellationDataFromGraphicItem(Guid outputPortId, IGraphicItem
                 var previousMeshVertexCount = package.MeshVertexCount;
 
                 //Todo Plane tessellation needs to be handled here vs in LibG currently
-                bool instancingEnabled = DynamoModel.FeatureFlags.CheckFeatureFlag<bool>("graphics-primitive-instancing", false);
+                bool instancingEnabled = DynamoModel.FeatureFlags?.CheckFeatureFlag<bool>("graphics-primitive-instancing", false) ?? false;
                 if (graphicItem is Plane plane)
                 {
                     CreatePlaneTessellation(package, plane);

src/DynamoCoreWpf/ViewModels/Menu/PreferencesViewModel.cs

@@ -1114,7 +1114,7 @@ public bool NodeAutocompleteMachineLearningIsBeta
         {
             get
             {
-                return DynamoModel.FeatureFlags.CheckFeatureFlag("NodeAutocompleteMachineLearningIsBeta", false);
+                return DynamoModel.FeatureFlags?.CheckFeatureFlag("NodeAutocompleteMachineLearningIsBeta", false) ?? false;
             }
         }
 

src/DynamoCoreWpf/ViewModels/Watch3D/HelixWatch3DViewModel.cs

@@ -1853,7 +1853,7 @@ internal virtual void AggregateRenderPackages(IEnumerable<HelixRenderPackage> pa
                         //If we have any line geometry that was not associated with an instance,
                         //remove the previously added line data from the render package so the remaining lines can be added to the scene.
                         if (rp.LineVertexRangesAssociatedWithInstancing.Any() 
-                            && DynamoModel.FeatureFlags.CheckFeatureFlag<bool>("graphics-primitive-instancing", false))
+                            && DynamoModel.FeatureFlags?.CheckFeatureFlag<bool>("graphics-primitive-instancing", false) == true)
                         {
                             //For each range of line vertices add the line data and instances to the scene
                             var j = 0;
@@ -1946,7 +1946,7 @@ internal virtual void AggregateRenderPackages(IEnumerable<HelixRenderPackage> pa
                     //If we have any mesh geometry that was not associated with an instance, remove the previously added
                     //mesh data from the render package so the remaining mesh can be added to the scene.
                     if (rp.MeshVertexRangesAssociatedWithInstancing.Any() 
-                        && DynamoModel.FeatureFlags.CheckFeatureFlag<bool>("graphics-primitive-instancing", false))
+                        && DynamoModel.FeatureFlags?.CheckFeatureFlag<bool>("graphics-primitive-instancing", false) == true)
                     {
                         //For each range of mesh vertices add the mesh data and instances to the scene
                         var j = 0;

src/DynamoCoreWpf/Views/Core/DynamoView.xaml.cs

@@ -1307,19 +1307,6 @@ private void DynamoView_Loaded(object sender, EventArgs e)
                 this.Deactivated += (s, args) => { HidePopupWhenWindowDeactivated(null); };
             }
             loaded = true;
-
-
-            //The following code illustrates use of FeatureFlagsManager.
-            //safe to remove.
-            if (DynamoModel.FeatureFlags != null)
-            {
-                CheckTestFlags();
-            }
-            //if feature flags is not yet initialized, subscribe to the event and wait.
-            else
-            {
-                DynamoUtilities.DynamoFeatureFlagsManager.FlagsRetrieved += CheckTestFlags;
-            }            
         }
 
         private void GuideFlowEvents_GuidedTourStart(GuidedTourStateEventArgs args)
@@ -1333,42 +1320,12 @@ private void GuideFlowEvents_GuidedTourStart(GuidedTourStateEventArgs args)
         /// <summary>
         /// Close Popup when the Dynamo window is not in the foreground.
         /// </summary>
-
         private void HidePopupWhenWindowDeactivated(object obj)
         {
             var workspace = this.ChildOfType<WorkspaceView>();
             if (workspace != null)
                 workspace.HideAllPopUp(obj);
         }
-        /// <summary>
-        /// check some test flags from launch darkly.
-        /// code is safe to remove at any time.
-        /// </summary>
-        private void CheckTestFlags()
-        {
-            if (!DynamoModel.IsTestMode)
-            {
-                //feature flag test.
-                if (DynamoModel.FeatureFlags?.CheckFeatureFlag<bool>("EasterEggIcon1", false) == true)
-                {
-                    dynamoViewModel.Model.Logger.Log("EasterEggIcon1 is true from view");
-                }
-                else
-                {
-                    dynamoViewModel.Model.Logger.Log("EasterEggIcon1 is false from view");
-                }
-
-                if (DynamoModel.FeatureFlags?.CheckFeatureFlag<string>("EasterEggMessage1", "NA") is string ffs && ffs != "NA")
-                {
-                    dynamoViewModel.Model.Logger.Log("EasterEggMessage1 is enabled from view");
-                    MessageBoxService.Show(this, ffs, "EasterEggMessage1", MessageBoxButton.OK, MessageBoxImage.Asterisk);
-                }
-                else
-                {
-                    dynamoViewModel.Model.Logger.Log("EasterEggMessage1 is disabled from view");
-                }
-            }
-        }
 
         private void TrackStartupAnalytics()
         {
@@ -1968,8 +1925,6 @@ private void WindowClosed(object sender, EventArgs e)
             this.dynamoViewModel.RequestReturnFocusToView -= OnRequestReturnFocusToView;
             this.dynamoViewModel.Model.WorkspaceSaving -= OnWorkspaceSaving;
             this.dynamoViewModel.Model.WorkspaceOpened -= OnWorkspaceOpened;
-            DynamoUtilities.DynamoFeatureFlagsManager.FlagsRetrieved -= CheckTestFlags;
-
             this.dynamoViewModel.RequestEnableShortcutBarItems -= DynamoViewModel_RequestEnableShortcutBarItems;
             this.dynamoViewModel.RequestExportWorkSpaceAsImage -= OnRequestExportWorkSpaceAsImage;
 

src/DynamoPackages/PackageManagerExtension.cs

@@ -337,9 +337,16 @@ internal bool CheckIfPackagesTargetOtherHosts(IEnumerable<PackageVersion> newPac
         {
             // determine if any of the packages are targeting other hosts
             var containsPackagesThatTargetOtherHosts = false;
-
-            // Known hosts
-            var knownHosts = PackageManagerClient.GetKnownHosts();
+            //fallback list of hosts as of 9/8/23
+            IEnumerable<string> knownHosts =  new List<string> { "Revit", "Civil 3D", "Alias", "Advance Steel", "FormIt" };
+            //if analytics is disabled, treat this as network black out
+            //and don't ask dpm for known hosts.
+            //TODO we currently use this like a no network comms flags - work on introducing a new flag or renaming this flag.
+            if (!Dynamo.Logging.Analytics.DisableAnalytics)
+            {
+                // Known hosts
+                knownHosts = PackageManagerClient.GetKnownHosts();
+            }
 
             // Sandbox, special case: Warn if any package targets only one known host
             if (String.IsNullOrEmpty(Host))

src/DynamoPackagesWPF/PackageManagerViewExtension.cs

@@ -157,7 +157,6 @@ private void RaisePackageHostNotifications(IEnumerable<Package> packages)
                         Resources.TitlePackageTargetOtherHost));
                 }
             }
-
         }
 
         private void RequestLoadLayoutSpecs(IEnumerable<Package> packages)

src/NodeServices/Analytics.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 
 namespace Dynamo.Logging
 {
@@ -7,6 +7,7 @@ namespace Dynamo.Logging
     /// </summary>
     public static class Analytics
     {
+        //TODO we currently use this like a no network comms flags - work on introducing a new flag or renaming this flag.
         /// <summary>
         /// Disables all analytics collection for the lifetime of the process.
         /// To ensure that no analytics get through, please set set this flag to false before the DynamoModel is constructed.

src/Notifications/NotificationCenterController.cs

@@ -92,10 +92,11 @@ internal NotificationCenterController(DynamoView view, DynamoLogger dynLogger)
                 VerticalOffset = notificationPopupVerticalOffset
             };
             logger = dynLogger;
-            
+
             // If user turns on the feature, they will need to restart Dynamo to see the count
             // This ensures no network traffic when Notification center feature is turned off
-            if (dynamoViewModel.PreferenceSettings.EnableNotificationCenter) 
+            //TODO we currently use DisableAnalytics like a no network comms flags - work on introducing a new flag or renaming this flag.
+            if (dynamoViewModel.PreferenceSettings.EnableNotificationCenter && !Dynamo.Logging.Analytics.DisableAnalytics ) 
             {               
                 InitializeBrowserAsync();
                 RequestNotifications();

test/DynamoCoreWpfTests/PackageManagerViewExtensionTests.cs

@@ -271,6 +271,7 @@ void PackageManagerViewExtensionTests_NotificationLogged(NotificationMessage obj
                 count = count + 1;
             }
         }
+
         [Test]
         public void PackageManagerViewExtesion_SendsNotificationForPackagesThatTargetDifferentHost_AtLatePackageLoad()
         {