Merge pull request #330 from brizental/1699080-telemetry-agent

Bug 1699080 - Change User-Agent header to X-Telemetry-Agent header

Author: Beatriz Rizental

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 [Full changelog](https://github.com/mozilla/glean.js/compare/v0.13.0...main)
 
+* [#313](https://github.com/mozilla/glean.js/pull/313): Send Glean.js version and platform information on X-Telemetry-Agent header instead of User-Agent header.
 # v0.13.0 (2021-05-18)
 
 [Full changelog](https://github.com/mozilla/glean.js/compare/v0.12.0...v0.13.0)

glean/src/core/upload/index.ts

@@ -133,7 +133,7 @@ class PingUploader implements PingsDatabaseObserver {
       "Date": (new Date()).toISOString(),
       "X-Client-Type": "Glean.js",
       "X-Client-Version": GLEAN_VERSION,
-      "User-Agent": `Glean/${GLEAN_VERSION} (JS on ${await this.platformInfo.os()})`
+      "X-Telemetry-Agent": `Glean/${GLEAN_VERSION} (JS on ${await this.platformInfo.os()})`
     };
 
     return {

glean/tests/core/upload/index.spec.ts

@@ -214,10 +214,10 @@ describe("PingUploader", function() {
     assert.strictEqual(appId, Context.applicationId);
 
     assert.ok("Date" in headers);
-    assert.ok("User-Agent" in headers);
+    assert.ok("Content-Length" in headers);
     assert.ok("Content-Type" in headers);
     assert.ok("X-Client-Type" in headers);
     assert.ok("X-Client-Version" in headers);
-    assert.ok("Content-Length" in headers);
+    assert.ok("X-Telemetry-Agent" in headers);
   });
 });