Fix for saving login details in some edge cases.

Martin Noreke · Martin Noreke · commit a8a675099041 · 2015-03-03T10:25:44.000-07:00
diff --git a/Rally.RestApi/Auth/ApiAuthManager.cs b/Rally.RestApi/Auth/ApiAuthManager.cs
@@ -703,7 +703,6 @@ private void UpdateAuthenticationState()
 				{
 					case RallyRestApi.AuthenticationResult.Authenticated:
 						AuthenticationStateChange.Invoke(Api.AuthenticationState, Api);
-						LoginDetails.SaveToDisk();
 						break;
 					case RallyRestApi.AuthenticationResult.PendingSSO:
 					case RallyRestApi.AuthenticationResult.NotAuthorized:
@@ -713,6 +712,9 @@ private void UpdateAuthenticationState()
 						throw new NotImplementedException();
 				}
 			}
+
+			if (Api.AuthenticationState == RallyRestApi.AuthenticationResult.Authenticated)
+				LoginDetails.SaveToDisk();
 		}
 		#endregion
 

Original file line number	Diff line number	Diff line change
`@@ -703,7 +703,6 @@ private void UpdateAuthenticationState()`
`703`	`703`	`{`
`704`	`704`	`case RallyRestApi.AuthenticationResult.Authenticated:`
`705`	`705`	`AuthenticationStateChange.Invoke(Api.AuthenticationState, Api);`
`706`		`- LoginDetails.SaveToDisk();`
`707`	`706`	`break;`
`708`	`707`	`case RallyRestApi.AuthenticationResult.PendingSSO:`
`709`	`708`	`case RallyRestApi.AuthenticationResult.NotAuthorized:`
`@@ -713,6 +712,9 @@ private void UpdateAuthenticationState()`
`713`	`712`	`throw new NotImplementedException();`
`714`	`713`	`}`
`715`	`714`	`}`
	`715`	`+`
	`716`	`+ if (Api.AuthenticationState == RallyRestApi.AuthenticationResult.Authenticated)`
	`717`	`+ LoginDetails.SaveToDisk();`
`716`	`718`	`}`
`717`	`719`	`#endregion`
`718`	`720`