Reduce LongPolling timeout to accommodate Cloudflare's timeout (#1368)
* Reduce LongPolling timeout to accommodate Cloudflare's timeout
This commit is contained in:
parent
bc3e15380c
commit
e214d5cdfa
|
|
@ -6,6 +6,6 @@ namespace Microsoft.AspNetCore.Sockets
|
||||||
{
|
{
|
||||||
public class LongPollingOptions
|
public class LongPollingOptions
|
||||||
{
|
{
|
||||||
public TimeSpan PollTimeout { get; set; } = TimeSpan.FromSeconds(110);
|
public TimeSpan PollTimeout { get; set; } = TimeSpan.FromSeconds(90);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -101,5 +101,12 @@ namespace Microsoft.AspNetCore.Sockets.Tests
|
||||||
var payload = ms.ToArray();
|
var payload = ms.ToArray();
|
||||||
Assert.Equal("Hello World", Encoding.UTF8.GetString(payload));
|
Assert.Equal("Hello World", Encoding.UTF8.GetString(payload));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public void CheckLongPollingTimeoutValue()
|
||||||
|
{
|
||||||
|
var options = new HttpSocketOptions();
|
||||||
|
Assert.Equal(options.LongPolling.PollTimeout, TimeSpan.FromSeconds(90));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue