refactor(core): drop unnecessary assignment of HOSTNAME (open-telemet…

…ry#4421) * fix(core): drop unnecessary assignemnt of HOSTNAME * fix(changelog): add changelog entry * Update CHANGELOG.md * fix(changelog): move entry to internal
Zirak · Sep 14, 2024 · 612c522 · 612c522
1 parent 5f7acf2
commit 612c522
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -20,6 +20,7 @@ For experimental package changes, see the [experimental CHANGELOG](experimental/
 ### :house: (Internal)
 
 * chore(opentelemetry-context-zone-peer-dep): support zone.js ^v0.13.0 [#4320](/~https://github.com/open-telemetry/opentelemetry-js/pull/4320)
+* refactor(core): drop unnecessary assignment of HOSTNAME [#4421](/~https://github.com/open-telemetry/opentelemetry-js/pull/4421) @pichlermarc
 
 ## 1.20.0
 

diff --git a/packages/opentelemetry-core/src/platform/node/environment.ts b/packages/opentelemetry-core/src/platform/node/environment.ts
@@ -14,7 +14,6 @@
  * limitations under the License.
  */
 
-import * as os from 'os';
 import {
   DEFAULT_ENVIRONMENT,
   ENVIRONMENT,
@@ -27,11 +26,5 @@ import {
  */
 export function getEnv(): Required<ENVIRONMENT> {
   const processEnv = parseEnvironment(process.env as RAW_ENVIRONMENT);
-  return Object.assign(
-    {
-      HOSTNAME: os.hostname(),
-    },
-    DEFAULT_ENVIRONMENT,
-    processEnv
-  );
+  return Object.assign({}, DEFAULT_ENVIRONMENT, processEnv);
 }