ensuring monitor to let all other threads go (#66)

artmasa · web-flow · commit 88832dba9950 · 2024-08-19T11:43:07.000-05:00
after the first metadata generation happens for an individual function no more blocking should happen and it's safe for all awaiting threads to be released.
diff --git a/src/isolated/Metadata/FunctionsAuthorizationMetadataMiddleware.cs b/src/isolated/Metadata/FunctionsAuthorizationMetadataMiddleware.cs
@@ -57,7 +57,7 @@ private async Task RegisterHttpTriggerAuthorizationAsync(FunctionContext context
             // is thread-safe on a per function basis.
             // Ensuring key is interned before entering monitor since key is compared as object
             var monitorKey = string.Intern($"famm:{context.FunctionId}");
-            await KeyedMonitor.EnterAsync(monitorKey);
+            await KeyedMonitor.EnterAsync(monitorKey, unblockOnFirstExit: true);
 
             try
             {

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ private async Task RegisterHttpTriggerAuthorizationAsync(FunctionContext context`
`57`	`57`	`// is thread-safe on a per function basis.`
`58`	`58`	`// Ensuring key is interned before entering monitor since key is compared as object`
`59`	`59`	`var monitorKey = string.Intern($"famm:{context.FunctionId}");`
`60`		`- await KeyedMonitor.EnterAsync(monitorKey);`
	`60`	`+ await KeyedMonitor.EnterAsync(monitorKey, unblockOnFirstExit: true);`
`61`	`61`
`62`	`62`	`try`
`63`	`63`	`{`