[email protected] 🌶️ README upd & SSR fix

siberiacancode · Feb 21, 2024 · 2842185 · 2842185
2 parents fba14db + b86268c
commit 2842185
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 8 deletions.
diff --git a/README.md b/README.md
@@ -61,8 +61,8 @@ The component also supports many properties for more specific work:
 | mainCamera                | boolean  | false        | should use a main camera (requires Navigator.mediaDevices.enumerateDevices)             |
 | frontCamera               | boolean  | false        | should use a front camera (MediaTrackConstraints['facingFront'] === 'user')             |
 | applyConstraints          | boolean  | false        | should new constraints be applied to the media stream                                   |
-| cameraResolutionType      | string   |              | video track resolution size - 'UHD' | 'QHD' | 'FHD' | 'HD'                              |
-| cameraResolutionMode      | string   | 'ideal'      | video track resolution mode - 'min' | 'max' | 'ideal' | 'exact'                         |
+| cameraResolutionType      | string   |              | video track resolution size - `('UHD' \| 'QHD' \| 'FHD' \| 'HD')`                       |
+| cameraResolutionMode      | string   | 'ideal'      | video track resolution mode - `('min' \| 'max' \| 'ideal' \| 'exact')`                  |
 | audioConstraints          | object   |              | audio track constraints - MediaStreamConstraints['audio']                               |
 | videoConstraints          | object   |              | video track constraints - MediaStreamConstraints['video']                               |
 | requestTimeLimit          | number   |              | limiting the media stream request by time                                               |

diff --git a/src/utils/helpers/stream/getUserMedia.ts b/src/utils/helpers/stream/getUserMedia.ts
@@ -3,10 +3,11 @@ import { hasGetUserMedia } from './hasGetUserMedia';
 // ✅ important
 // Implementations for each browser need to be supported
 export const getUserMediaFunction =
-  navigator.getUserMedia ||
-  navigator.webkitGetUserMedia ||
-  navigator.mozGetUserMedia ||
-  navigator.msGetUserMedia;
+  typeof window !== 'undefined' &&
+  (navigator.getUserMedia ||
+    navigator.webkitGetUserMedia ||
+    navigator.mozGetUserMedia ||
+    navigator.msGetUserMedia);
 
 // ✅ important
 // Polyfill to support legacy web-api

diff --git a/src/utils/helpers/stream/stopMediaStream.ts b/src/utils/helpers/stream/stopMediaStream.ts
@@ -1,4 +1,6 @@
-export const stopMediaStream = (stream: MediaStreamTrack | MediaStream) => {
+export const stopMediaStream = (stream?: MediaStreamTrack | MediaStream) => {
+  if (!stream) return;
+
   if (stream instanceof MediaStreamTrack) {
     stream.stop();
     return;

diff --git a/src/utils/hooks/useMediaStream.ts b/src/utils/hooks/useMediaStream.ts
@@ -95,7 +95,6 @@ export const useMediaStream = ({
 
     return () => {
       handlerRef.current.stop?.(runningStream);
-      if (!runningStream) return;
       stopMediaStream(runningStream);
     };
   }, [