Resource Detectors prorogate bugs that have long been fixed by opentelemtry-js

[davemyers-dev]

The resource detectors that were copied into this library, seemingly to work around an sync issue which was also fixed a year ago, continue to propagate issues that open telemetry has already fixed in their source.

All of Otel's source detectors have been updated since they were copied.

Ex of a bug only present in this wrapper as a result:
open-telemetry/opentelemetry-js#3295

Otel also made changes to allow synchronous detectors in open-telemetry/opentelemetry-js#2912. This may mean that these do not need to be copied into this library at all.

Proposals:
Copied detectors should be removed from this library
or
Copied code should be properly maintained and updated with latest from source

[seemk]

With #925 detectors have been switched to upstream ones (with the exception of container detector).

However do note that spaces in resource attribute values need to be encoded as %20, see open-telemetry/opentelemetry-js#3131

E.g.
export OTEL_RESOURCE_ATTRIBUTES='service.name="foo bar"'
needs to be written as
export OTEL_RESOURCE_ATTRIBUTES='service.name="foo%20bar"'