Async version of Ping?

[ozziepeeps]

Hi, I am streaming images in through ASP.NET Core and directly into S3 and as part of that process I make use of a decorator pattern around a Stream in order to run the stream through Magick.NET. One of the functions I perform is Ping in order to extract the image width and height, but due to this using synchronous I/O under the covers, it forces my hand to enable synchronous IO for Kestrel.

This workaround works fine, but I wonder if you might consider a PingAsync method that performs asynchronous I/O, or whether you have any other suggestions?

Thanks for a great library!

[dlemstra]

Thanks for asking about this. The current async approach is to read all the data asynchronous and then read the data from the byte array. And that was the reason that I did not implement an asynchronous version of the Ping method. But after a chat with GitHub Copilot I found a possible different way to do this. Not sure when I will make time for this but I will be going to check if that solution is going to work.

[ozziepeeps]

Thanks for the reply! Your suggested workaround of asynchrously reading into memory and then reading data from a byte array won’t work for me due to the size of the imagery (and therefore memory pressure) I have to deal with.

Excited to see what you (and Copilot!) come up with!

[dlemstra]

I just pushed a patch that adds PingAsync to MagickImage and MagickImageCollection. This will become available in the next release.

[ozziepeeps]

Awesome, thank you sir! The commits dea4378 and b1da461 look good to me. Excited to try this once released. I appreciate you!