Merge branch 'master' into github-enterprise

# Conflicts:
#	src/GitHub.Api/Application/ApiClient.cs
#	src/GitHub.Api/Authentication/LoginManager.cs

Author: StanleyGoldman

docs/contributing/how-to-build.md

@@ -13,7 +13,7 @@ This repository is LFS-enabled. To clone it, you should use a git client that su
 
 ### MacOS
 
-- Mono 4.x required. You can install it via brew with `brew tap shana/mono && brew install mono@4.8`
+- [Mono 4.x](https://download.mono-project.com/archive/4.8.1/macos-10-universal/) required. You can install it via brew with `brew tap shana/mono && brew install mono@4.8`
   - Mono 5.x will not work
 - `UnityEngine.dll` and `UnityEditor.dll`.
   - If you've installed Unity in the default location of `/Applications/Unity`, the build will be able to reference these DLLs automatically. Otherwise, you'll need to copy these DLLs from `[Unity installation path]/Unity.app/Contents/Managed` into the `lib` directory in order for the build to work

src/GitHub.Api/Application/ApiClient.cs

@@ -441,7 +441,7 @@ private GitHubUser GetValidatedGitHubUser()
                 {
                     var login = ret.Output[1];
 
-                    if (login != Connection.Username)
+                    if (!string.Equals(login, Connection.Username, StringComparison.InvariantCultureIgnoreCase))
                     {
                         logger.Trace("LoadKeychainInternal: Api username does not match");
                         throw new TokenUsernameMismatchException(Connection.Username, login);

src/GitHub.Api/Cache/CacheInterfaces.cs

@@ -40,6 +40,7 @@ public interface IManagedCache
 
         bool ValidateData();
         void InvalidateData();
+        void ResetInvalidation();
 
         DateTimeOffset LastUpdatedAt { get; }
         CacheType CacheType { get; }
@@ -91,7 +92,7 @@ public interface IBranchCache : IManagedCache
         ILocalConfigBranchDictionary LocalConfigBranches { get; }
         IRemoteConfigBranchDictionary RemoteConfigBranches { get; }
         IConfigRemoteDictionary ConfigRemotes { get; }
-        
+
         void SetRemotes(Dictionary<string, ConfigRemote> remoteConfigs, Dictionary<string, Dictionary<string, ConfigBranch>> configBranches, GitRemote[] gitRemotes, GitBranch[] gitBranches);
         void SetLocals(Dictionary<string, ConfigBranch> configBranches, GitBranch[] gitBranches);
     }

src/GitHub.Api/Git/Repository.cs

@@ -208,39 +208,50 @@ private void CacheHasBeenInvalidated(CacheType cacheType)
             switch (cacheType)
             {
                 case CacheType.Branches:
-                    repositoryManager?.UpdateBranches().Start();
+                    repositoryManager?.UpdateBranches().Catch(ex => InvalidationFailed(ex, cacheType)).Start();
                     break;
 
                 case CacheType.GitLog:
-                    repositoryManager?.UpdateGitLog().Start();
+                    repositoryManager?.UpdateGitLog().Catch(ex => InvalidationFailed(ex, cacheType)).Start();
                     break;
 
                 case CacheType.GitAheadBehind:
-                    repositoryManager?.UpdateGitAheadBehindStatus().Start();
+                    repositoryManager?.UpdateGitAheadBehindStatus().Catch(ex => InvalidationFailed(ex, cacheType)).Start();
                     break;
 
                 case CacheType.GitLocks:
                     if (CurrentRemote != null)
-                        repositoryManager?.UpdateLocks().Start();
+                    {
+                        repositoryManager?.UpdateLocks().Catch(ex => InvalidationFailed(ex, cacheType)).Start();
+                    }
                     break;
 
                 case CacheType.GitUser:
                     // user handles its own invalidation event
                     break;
 
                 case CacheType.RepositoryInfo:
-                    repositoryManager?.UpdateRepositoryInfo().Start();
+                    repositoryManager?.UpdateRepositoryInfo().Catch(ex => InvalidationFailed(ex, cacheType)).Start();
                     break;
 
                 case CacheType.GitStatus:
-                    repositoryManager?.UpdateGitStatus().Start();
+                    repositoryManager?.UpdateGitStatus().Catch(ex => InvalidationFailed(ex, cacheType)).Start();
                     break;
 
                 default:
                     throw new ArgumentOutOfRangeException(nameof(cacheType), cacheType, null);
             }
         }
 
+        private bool InvalidationFailed(Exception ex, CacheType cacheType)
+        {
+            Logger.Warning(ex, "Error invalidating {0}", cacheType);
+            var managedCache = cacheContainer.GetCache(cacheType);
+            managedCache.ResetInvalidation();
+            return false;
+        }
+
+
         private void RepositoryManagerOnCurrentBranchUpdated(ConfigBranch? branch, ConfigRemote? remote, string head)
         {
             taskManager.RunInUI(() =>
@@ -254,7 +265,7 @@ private void RepositoryManagerOnCurrentBranchUpdated(ConfigBranch? branch, Confi
                 name = null;
                 cloneUrl = null;
                 cacheContainer.RepositoryInfoCache.UpdateData(data);
-               
+
                 // force refresh of the Name and CloneUrl propertys
                 var n = Name;
             });
@@ -485,6 +496,7 @@ private void UpdateUserAndEmail()
             }
 
             gitClient.GetConfigUserAndEmail()
+                     .Catch(InvalidationFailed)
                      .ThenInUI((success, value) =>
                      {
                          if (success)
@@ -494,7 +506,15 @@ private void UpdateUserAndEmail()
                          }
                      }).Start();
         }
-        
+
+        private bool InvalidationFailed(Exception ex)
+        {
+            Logger.Warning(ex, "Error invalidating user cache");
+            var managedCache = cacheContainer.GetCache(CacheType.GitUser);
+            managedCache.ResetInvalidation();
+            return false;
+        }
+
         public string Name
         {
             get { return cacheContainer.GitUserCache.Name; }

src/UnityExtension/Assets/Editor/GitHub.Unity/ApplicationCache.cs

@@ -213,12 +213,14 @@ public void InvalidateData()
 
         private void Invalidate()
         {
-            if (!isInvalidating)
-            {
-                isInvalidating = true;
-                LastUpdatedAt = DateTimeOffset.MinValue;
-                CacheInvalidated.SafeInvoke(CacheType);
-            }
+            isInvalidating = true;
+            LastUpdatedAt = DateTimeOffset.MinValue;
+            CacheInvalidated.SafeInvoke(CacheType);
+        }
+
+        public void ResetInvalidation()
+        {
+            isInvalidating = false;
         }
 
         protected void SaveData(DateTimeOffset now, bool isChanged)
@@ -463,25 +465,25 @@ public void UpdateData(IRepositoryInfoCacheData data)
                 isUpdated = true;
             }
 
-            if (forcedInvalidation ||!Nullable.Equals(currentGitBranch, data.CurrentGitBranch))
+            if (forcedInvalidation || !Nullable.Equals(currentGitBranch, data.CurrentGitBranch))
             {
                 currentGitBranch = data.CurrentGitBranch ?? GitBranch.Default;
                 isUpdated = true;
             }
 
-            if (forcedInvalidation ||!Nullable.Equals(currentConfigRemote, data.CurrentConfigRemote))
+            if (forcedInvalidation || !Nullable.Equals(currentConfigRemote, data.CurrentConfigRemote))
             {
                 currentConfigRemote = data.CurrentConfigRemote ?? ConfigRemote.Default;
                 isUpdated = true;
             }
 
-            if (forcedInvalidation ||!Nullable.Equals(currentConfigBranch, data.CurrentConfigBranch))
+            if (forcedInvalidation || !Nullable.Equals(currentConfigBranch, data.CurrentConfigBranch))
             {
                 currentConfigBranch = data.CurrentConfigBranch ?? ConfigBranch.Default;
                 isUpdated = true;
             }
 
-            if (forcedInvalidation ||!String.Equals(currentHead, data.CurrentHead))
+            if (forcedInvalidation || !String.Equals(currentHead, data.CurrentHead))
             {
                 currentHead = data.CurrentHead;
                 isUpdated = true;

src/tests/IntegrationTests/CachingClasses.cs

@@ -109,6 +109,7 @@ abstract class ManagedCacheBase<T> : ScriptObjectSingleton<T> where T : class, I
         private DateTimeOffset? initializedAtValue;
 
         private bool isInvalidating;
+        protected bool forcedInvalidation;
 
         public event Action<CacheType> CacheInvalidated;
         public event Action<CacheType, DateTimeOffset> CacheUpdated;
@@ -134,13 +135,20 @@ public bool ValidateData()
 
         public void InvalidateData()
         {
-            if (!isInvalidating)
-            {
-                Logger.Trace("Invalidate");
-                isInvalidating = true;
-                LastUpdatedAt = DateTimeOffset.MinValue;
-                CacheInvalidated.SafeInvoke(CacheType);
-            }
+            forcedInvalidation = true;
+            Invalidate();
+        }
+
+        private void Invalidate()
+        {
+            isInvalidating = true;
+            LastUpdatedAt = DateTimeOffset.MinValue;
+            CacheInvalidated.SafeInvoke(CacheType);
+        }
+
+        public void ResetInvalidation()
+        {
+            isInvalidating = false;
         }
 
         protected void SaveData(DateTimeOffset now, bool isChanged)