Disable net traffic

[pinzart90]

To achieve network blackout this is what I had to do:

• DynamoSandbox started with disableAnalytics flag and notification + autocomplete disabled in UI.
• DefaultUpdateManager.DisableUpdates set to true
• changes in this PR

[QilongTang]

This code would not raise network traffic right? This is the code to check if the installed package is targetting other host. I think @zeusongit recently worked on it..

[pinzart90]

I commented it out because it did try to contact pacakage manager.
I could not find a way to disable through API or configs. @zeusongit any idea if there is a way to disable this ?

[QilongTang]

After reading the code, I can see why now. In function CheckIfPackagesTargetOtherHosts there is a call var knownHosts = PackageManagerClient.GetKnownHosts(); which will contact package manager

[QilongTang]

I think this solution is fine for now but we probably want to guard all of these code using the env variable or setting property on master branch.

[QilongTang]

Are there code changes related to DefaultUpdateManager.DisableUpdates set to true?

[pinzart90]

DisableUpdates

Only locally, to reflect the changes in the ALIAS repo
From the ALIAS repo:

class UpdateManagerConfig : IUpdateManagerConfiguration, IDisableUpdateConfig
...
        public bool DisableUpdates
        {
            get { return true; }
            set { }
        }