Script disables bfcache because of deprecated use of unload

[JasonUSC]

Describe the bug

This script disables bfcache because of its use of the deprecated unload event. Both unload and beforeuload are known to be unreliable, so why cause a performance hit and SEO ding on browsers?

window.addEventListener("beforeunload",(function(){e()}),!0),window.addEventListener("unload",(function()

Reproduction Steps?

https://developer.chrome.com/docs/web-platform/deprecating-unload

SDK Versions

https://global.oktacdn.com/okta-auth-js/7.14.1/okta-auth-js.min.js

Additional Information?

No response

[jaredperreault-okta]

It's unlikely this will be fixed any time soon. The usage of unload and beforeunload are coming from a dependency https://github.com/pubkey/broadcast-channel/blob/master/dist/lib/browser.js#L1992 and this repo has GH issues disabled.

As a work around you can likely override the addEventListener method to disable the ability to bind unload events altogether

      window.originalAddEventListener = window.addEventListener;
      window.addEventListener = (event, handler, opts) => {
        if (event === 'unload') {
           event = 'pagehide';      // or just return?
        }
        window.originalAddEventListener(event, handler, opts);
      }