Refactored code broke it

[rany2]

I reviewed https://github.com/anaclumos/bing-chat-for-all-browsers/blob/main/src/background.ts#L12-L13

Basically you set header.value to DESKTOP_UA and then check if it contains 'mobile', which will never happen because you set header.value to DESKTOP_UA.

[rany2]

Suggested change:

chrome.webRequest.onBeforeSendHeaders.addListener(
  (details) => {
    const { requestHeaders } = details
    if (!requestHeaders) return undefined
    const newHeaders = requestHeaders.map((header) => {
      if (header.name.toLowerCase() === 'user-agent') {
        if (header.value?.toLowerCase().includes('mobile'))
          header.value = MOBILE_UA;
        else
          header.value = DESKTOP_UA;
      }
      return header
    })
    return { requestHeaders: newHeaders }
  },
  { urls: ['*://*.bing.com/*'] },
  ['blocking', 'requestHeaders']
)

[anaclumos]

Oh, what a newbie mistake. Will issue a fix right away.

[rany2]

Thanks, I was confused for a second and thought it was my fault :)

[rany2]

Thanks!