Timeout not working as expected

I’m doing some testing on failure scenarios, and am testing what happens if the NATS server is stopped while my process is running.

I’m finding the timeout periods seem to be getting ignored, assuming I’ve configured it correctly.

Here is my reproduce code.

using NATS.Client;
using System;
using System.Diagnostics;
using System.Threading;
using System.Threading.Tasks;

namespace TestNats
{
    class Program
    {
        static void Main(string[] args)
        {
            Task.Run(() => Run());
            Console.ReadLine();
            
        }

        static async Task Run()
        {
            Console.WriteLine("Hello World!");
            var cf = new ConnectionFactory();
            var opts = ConnectionFactory.GetDefaultOptions();
            opts.Timeout = 1000;
            opts.AllowReconnect = false;
            var conn = cf.CreateConnection(opts);
            conn.SubscribeAsync("test", (p, a) => conn.Publish(a.Message.Reply, new byte[0]));
            await conn.RequestAsync("test", new byte[0], 1000);

            Console.WriteLine("Now stop nats server, you have 5 seconds");
            Thread.Sleep(5000);
            
            var s = new Stopwatch();
            s.Start();
            try
            {
                await conn.RequestAsync("test", new byte[0], 1000);
                s.Stop();
                Console.WriteLine(s.ElapsedMilliseconds / 1000 + "s");
            }
            catch (Exception ex)
            {
                s.Stop();
                Console.WriteLine(s.ElapsedMilliseconds / 1000 + "s");
                Console.WriteLine(ex.Message);
            }
        }
    }
}

Output:

Hello World!
Now stop nats server, you have 5 seconds
60s
Timeout occurred.

[chrismcv]

@ColinSullivan1 no problem - and thank for the quick response too.

I’ve tested your PR and it is giving the following output for my sample code.

Hello World!
Now stop nats server, you have 5 seconds
0s
Timeout occurred.

This seems good to me. Thanks for your help!

[ColinSullivan1]

@chrismcv, Great - thank you for testing! My local tests pass as well; I’ll merge the PR.