Merge pull request #1479 from github/fixes/pr-session-manager-initialized-exception

grokys · web-flow · commit 988bf190403c · 2018-02-12T16:18:10.000+01:00
Prevent PullRequestSessionManager.StatusChanged throwing
diff --git a/src/GitHub.InlineReviews/Services/PullRequestSessionManager.cs b/src/GitHub.InlineReviews/Services/PullRequestSessionManager.cs
@@ -236,7 +236,7 @@ async Task StatusChanged()
                 }
 
                 CurrentSession = session;
-                initialized.SetResult(null);
+                initialized.TrySetResult(null);
             }
             catch (Exception e)
             {

Original file line number	Diff line number	Diff line change
`@@ -236,7 +236,7 @@ async Task StatusChanged()`
`236`	`236`	`}`
`237`	`237`
`238`	`238`	`CurrentSession = session;`
`239`		`- initialized.SetResult(null);`
	`239`	`+ initialized.TrySetResult(null);`
`240`	`240`	`}`
`241`	`241`	`catch (Exception e)`
`242`	`242`	`{`