nats-io / nats.c

A C client for NATS
Apache License 2.0
384 stars 134 forks source link

Fixed download of NATS Server for latest/main #711

Closed kozlovic closed 8 months ago

kozlovic commented 8 months ago

The "latest" was not working and would always use the one from nats.c.deps. As for "main", I noticed lately that sometimes, we would get a binary of size 0, which would make all tests using the nats-server to fail. So make sure that we can run the server and get a valid version string. If not, repeat the test up to 5 times, otherwise fail the run.

Signed-off-by: Ivan Kozlovic ivan@synadia.com

kozlovic commented 8 months ago

@levb If you get a chance tomorrow (01/16/2024), could you review this one? Without this, the run that uses main will fail more often than not because the download of the binary fails, so I added few tries.