Environment::nodeId Should Throw SystemException When Node ID is 0 about poco HOT 3 CLOSED

EnvironmentImpl::nodeIdImpl() should be changed to also look for WiFi adapters if no Ethernet adapter was found.
And throw only if neither an Ethernet nor a WiFi adapter could be found.

from poco.

I think that this would work better if a SystemException was thrown when nodeIdImpl fails to set a value.

The documentation says if it can't find an ethernet adapter, but it should throw if it can not produce nodeId for any reason.

@andrewauclair please create a PR, and @matejk and myself will review

from poco.

Sounds good. One of my coworkers just happened to run into the Windows issue I mentioned. It looks like our Dell laptops only have wifi adapters and no ethernet adapters. The win32 nodeIdImpl only looks for ethernet adapters. Not sure what a good solution would be for this. Going to give it some thought.

from poco.