Skip to content

HostFactoryResolver - Increase default timeout to thirty seconds #61621

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
maryamariyan merged 8 commits into from
Nov 17, 2021
Merged

HostFactoryResolver - Increase default timeout to thirty seconds #61621

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
b977b58
patch fix
maryamariyan Nov 15, 2021
7535772
improve messaging
maryamariyan Nov 15, 2021
5c4a8c9
Adds env var options for setting timeout
maryamariyan Nov 16, 2021
26d3bb8
Add test for timeout
maryamariyan Nov 16, 2021
aacc7e1
Update src/libraries/Microsoft.Extensions.HostFactoryResolver/src/Hos…
maryamariyan Nov 16, 2021
8aae991
PR feedback - increased the timeout length
maryamariyan Nov 16, 2021
8326cfa
Update src/libraries/Microsoft.Extensions.HostFactoryResolver/tests/H…
maryamariyan Nov 16, 2021
0b2f189
Update src/libraries/Microsoft.Extensions.HostFactoryResolver/tests/T…
maryamariyan Nov 16, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Adds env var options for setting timeout
  • Loading branch information
@maryamariyan
maryamariyan committed Nov 16, 2021
commit 5c4a8c9064ec394324543c79c53e8d56b9e1e2e6
18 changes: 17 additions & 1 deletion src/libraries/Microsoft.Extensions.HostFactoryResolver/src/HostFactoryResolver.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,25 @@ internal sealed class HostFactoryResolver
public const string BuildWebHost = nameof(BuildWebHost);
public const string CreateWebHostBuilder = nameof(CreateWebHostBuilder);
public const string CreateHostBuilder = nameof(CreateHostBuilder);
private const string TimeoutEnvironmentKey = "DOTNET_HOST_FACTORY_RESOLVER_DEFAULT_TIMEOUT_IN_SECONDS";

// The amount of time we wait for the diagnostic source events to fire
private static readonly TimeSpan s_defaultWaitTimeout = Debugger.IsAttached ? Timeout.InfiniteTimeSpan : TimeSpan.FromMinutes(1);
private static readonly TimeSpan s_defaultWaitTimeout = SetupDefaultTimout();

private static TimeSpan SetupDefaultTimout()
{
if (Debugger.IsAttached)
{
return Timeout.InfiniteTimeSpan;
}

if (uint.TryParse(Environment.GetEnvironmentVariable(TimeoutEnvironmentKey), out uint timeoutInSeconds))
{
return TimeSpan.FromSeconds((int)timeoutInSeconds);
}

return TimeSpan.FromSeconds(30);
}

public static Func<string[], TWebHost>? ResolveWebHostFactory<TWebHost>(Assembly assembly)
{
Expand Down