On Fri, 15 Oct 2021 10:01:19 GMT, Daniel Jeliński <d...@openjdk.java.net> wrote:

> It happens occasionally, as evidenced by JDK-8217298, JDK-8046500 (fixed), 
> JDK-8165665, JDK-8066931, JDK-8057900, JDK-8065559(closed), JDK-8040229, 
> JDK-8065078(fixed), JDK-8068597(fixed), numerous reports on Google (search 
> for "error IP Helper Library"), and an ancient note found in my company's 
> codebase, possibly no longer relevant.

So these Errors do actually happen - thanks for the confirmation, and double 
thanks for the archeology research. This is very useful to this discussion.

I see that some of the issue you mentioned are suggesting some fixes - like - 
for instance JDK-8165665. 
It may be that we need to examine each of these on a case-by-case basis. It 
seems that `GetAdaptersAddresses`, for instance, has more error codes than 
those we specifically handle - and maybe we need more specific error handling 
for some of these. 
Maybe throwing `SocketException`, as you suggest, is appropriate for some of 
these cases. 
Maybe just skipping the interface would be the right choice for others?

If some conditions are not supposed to happen - then I still think that 
throwing `InternalError` is more appropriate.
I would be afraid that blindly changing all `Error` into `SocketException` 
would be hiding issues under the carpet, as they would appear as a regular 
failure...

-------------

PR: https://git.openjdk.java.net/jdk/pull/5956

Reply via email to