fix: update HTTP headers sent to otel endpoints

Author: jkasten2

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/core/internal/http/impl/HttpClient.kt

@@ -26,6 +26,9 @@ import java.net.UnknownHostException
 import java.util.Scanner
 import javax.net.ssl.HttpsURLConnection
 
+internal const val HTTP_SDK_VERSION_HEADER_KEY = "SDK-Version"
+internal val HTTP_SDK_VERSION_HEADER_VALUE = "onesignal/android/${OneSignalUtils.sdkVersion}"
+
 internal class HttpClient(
     private val _connectionFactory: IHttpConnectionFactory,
     private val _prefs: IPreferencesService,
@@ -131,7 +134,7 @@ internal class HttpClient(
                     con.useCaches = false
                     con.connectTimeout = timeout
                     con.readTimeout = timeout
-                    con.setRequestProperty("SDK-Version", "onesignal/android/" + OneSignalUtils.sdkVersion)
+                    con.setRequestProperty(HTTP_SDK_VERSION_HEADER_KEY, HTTP_SDK_VERSION_HEADER_VALUE)
 
                     if (OneSignalWrapper.sdkType != null && OneSignalWrapper.sdkVersion != null) {
                         con.setRequestProperty("SDK-Wrapper", "onesignal/${OneSignalWrapper.sdkType}/${OneSignalWrapper.sdkVersion}")

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/debug/internal/logging/otel/OneSignalOpenTelemetry.kt

@@ -3,6 +3,8 @@ package com.onesignal.debug.internal.logging.otel
 import android.os.Build
 import androidx.annotation.RequiresApi
 import com.onesignal.core.internal.config.ConfigModelStore
+import com.onesignal.core.internal.http.impl.HTTP_SDK_VERSION_HEADER_KEY
+import com.onesignal.core.internal.http.impl.HTTP_SDK_VERSION_HEADER_VALUE
 import com.onesignal.debug.internal.logging.otel.attributes.OneSignalOtelFieldsPerEvent
 import com.onesignal.debug.internal.logging.otel.attributes.OneSignalOtelFieldsTopLevel
 import com.onesignal.debug.internal.logging.otel.config.OtelConfigCrashFile
@@ -71,7 +73,8 @@ internal class OneSignalOpenTelemetryRemote(
     IOneSignalOpenTelemetryRemote {
     val extraHttpHeaders by lazy {
         mapOf(
-            "OS-App-Id" to _configModelStore.model.appId,
+            "X-OneSignal-App-Id" to _configModelStore.model.appId,
+            HTTP_SDK_VERSION_HEADER_KEY to HTTP_SDK_VERSION_HEADER_VALUE,
             "x-honeycomb-team" to "", // TODO: REMOVE
         )
     }