fix: Check container response null pointer

testcontainers / testcontainers-dotnet

A library to support tests with throwaway instances of Docker containers for all compatible .NET Standard versions.

https://dotnet.testcontainers.org

MIT License

3.73k stars 262 forks source link

fix: Check container response null pointer #1024

Closed HofmeisterAn closed 10 months ago

HofmeisterAn commented 10 months ago

What does this PR do?

The PR adds a null pointer check.

Why is it important?

With the recent adjustments, getting the container response will no longer throw an exception. Instead, it will return a null value. The Exists() did not check for a null value and threw an exception. This bug was introduced in the development version.

Related issues

netlify[bot] commented 10 months ago

Deploy Preview for testcontainers-dotnet ready!

Name	Link
Latest commit	942ecbe44fda54d006ce459401fcbd2d004c18f4
Latest deploy log	https://app.netlify.com/sites/testcontainers-dotnet/deploys/652a3a798a4f360008437422
Deploy Preview	https://deploy-preview-1024--testcontainers-dotnet.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.