fix handle client exception

[krushnarout]

omi/app/lib/backend/http/http_pool_manager.dart

Line 84 in 020ad86

throw lastError!;

Network request fails with ClientException with SocketException after retries complete and the error is thrown at throw lastError!;

[gemini-code-assist]

Code Review

This pull request effectively resolves an issue where ClientException was not being handled by the retry logic in HttpPoolManager. The addition of a specific catch block for http.ClientException ensures these errors are now retried, enhancing the application's network resilience. Furthermore, the change from throw lastError! to throw lastError ?? Exception(...) is a good defensive improvement, preventing potential null reference errors if the retry loop doesn't execute. The changes are correct and improve the robustness of the code.

[beastoin]

You mean the lastError was null?

[krushnarout]

You mean the lastError was null?

yes, the retry loop could exit with lastError == null, causing a crash when throw lastError! was reached