From 0b2be345eca2db17ca977584a4114ab5aae6be71 Mon Sep 17 00:00:00 2001 From: nick-delirium Date: Fri, 3 Nov 2023 11:52:18 +0100 Subject: [PATCH] fix(tracker): fix network req hs case sensitivity --- tracker/tracker/CHANGELOG.md | 4 ++++ tracker/tracker/package.json | 2 +- tracker/tracker/src/main/modules/Network/networkMessage.ts | 7 +++++-- tracker/tracker/src/main/modules/network.ts | 2 +- 4 files changed, 11 insertions(+), 4 deletions(-) diff --git a/tracker/tracker/CHANGELOG.md b/tracker/tracker/CHANGELOG.md index 6fa70e5e5..c91e6a266 100644 --- a/tracker/tracker/CHANGELOG.md +++ b/tracker/tracker/CHANGELOG.md @@ -1,3 +1,7 @@ +# 10.0.2 + +- fix default ignore headers + # 10.0.1 - network proxy api is now default turned on diff --git a/tracker/tracker/package.json b/tracker/tracker/package.json index 098b5ff28..8bc3c6fba 100644 --- a/tracker/tracker/package.json +++ b/tracker/tracker/package.json @@ -1,7 +1,7 @@ { "name": "@openreplay/tracker", "description": "The OpenReplay tracker main package", - "version": "10.0.1", + "version": "10.0.2", "keywords": [ "logging", "replay" diff --git a/tracker/tracker/src/main/modules/Network/networkMessage.ts b/tracker/tracker/src/main/modules/Network/networkMessage.ts index b232c1c62..98e1c3a76 100644 --- a/tracker/tracker/src/main/modules/Network/networkMessage.ts +++ b/tracker/tracker/src/main/modules/Network/networkMessage.ts @@ -94,7 +94,10 @@ export default class NetworkMessage { } isHeaderIgnored(key: string) { - if (Array.isArray(this.ignoredHeaders)) return this.ignoredHeaders.includes(key) - return this.ignoredHeaders + if (Array.isArray(this.ignoredHeaders)) { + return this.ignoredHeaders.map((k) => k.toLowerCase()).includes(key.toLowerCase()) + } else { + return this.ignoredHeaders + } } } diff --git a/tracker/tracker/src/main/modules/network.ts b/tracker/tracker/src/main/modules/network.ts index f86e87a1a..bb440eda2 100644 --- a/tracker/tracker/src/main/modules/network.ts +++ b/tracker/tracker/src/main/modules/network.ts @@ -63,7 +63,7 @@ export default function (app: App, opts: Partial = {}) { const options: Options = Object.assign( { failuresOnly: false, - ignoreHeaders: ['Cookie', 'Set-Cookie', 'Authorization'], + ignoreHeaders: ['cookie', 'set-cookie', 'authorization'], capturePayload: false, sessionTokenHeader: false, captureInIframes: true,