fix: navigator is not defined.

[thePeeyush]

Fix Issue #1165: Navigator is not defined.

Cause: Today many people use NextJS for building webapps. But when they importing Peer from peerjs they get this error because when webpack compile it in server side, navigator is not present there.

Issue: link

Changes Made:

In peerjs/dist/bundel.mjs : I made change in this file as given below literally remove the error from compiling and building in simple import without using useeffect or dynamic import. Only 'use client' is defined at top.

if (typeof navigator !== 'undefined') {
            this.isIOS = ["iPad", "iPhone", "iPod"].includes(navigator.platform)
        }
        else this.isIOS = false;

So this means, checking the navigator in support.ts is able to solve this problem.

Now : readonly isIOS = ["iPad", "iPhone", "iPod"].includes(navigator.platform);

Change to : readonly isIOS = typeof navigator !== "undefined" ? (["iPad", "iPhone", "iPod"].includes(navigator.platform)) : false;;

The navigator.platform only works when navigator is available else isIOS assigned false.

Reviewer Requests:

@jonasgloning : Please review the changes and provide feedback.

[irgalamarr]

Hi @thePeeyush!

Thank you for the PR! I went over all previous issues and discussions about the problem. Upon reviewing the codebase, I noticed several other window and navigator references, which could potentially impact SSR in a similar manner.

Did I get it right from the discussion that using a local build of PeerJS with these changes was all you needed for a smooth SSR build for your projects? After, in runtime, it works correct?

[thePeeyush]

Hello @irgalamarr mam ,

Thanks for the discussion, I built this lib by modifying these changes and made a new npm package called airsharejs for testing. And it works completely fine without encountering any errors in vercel build and local build.

With PeerJS:

With airsharejs ( modified peerjs):

And it worked correctly in runtime. I made a file-sharing app in NextJS with airsharejs pkg and it runs smoothly in all browsers. You can checkout
Link: https://air0.vercel.app/

So it helps all NextJS devs to make apps with peerjs, I can check for build on other SSR frameworks if you want?

[github-actions]

🎉 This PR is included in version 1.5.3 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀