Skip to content

Commit

Permalink
chore: added handling for no mediastream and no mediastreamtrack
Browse files Browse the repository at this point in the history
  • Loading branch information
malmen237 committed May 10, 2024
1 parent eb4f3cc commit b28a3b1
Showing 1 changed file with 24 additions and 1 deletion.
25 changes: 24 additions & 1 deletion src/components/production-line/use-rtc-connection.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import {
useRef,
useState,
} from "react";
import { useNavigate } from "react-router-dom";
import { noop } from "../../helpers";
import { API } from "../../api/api.ts";
import { TJoinProductionOptions } from "./types.ts";
Expand All @@ -28,6 +29,7 @@ type TEstablishConnection = {
sessionId: string;
dispatch: Dispatch<TGlobalStateAction>;
setAudioElements: Dispatch<SetStateAction<HTMLAudioElement[]>>;
setNoStreamError: (input: boolean) => void;
};

type TAttachAudioStream = {
Expand All @@ -50,6 +52,7 @@ const establishConnection = ({
sessionId,
dispatch,
setAudioElements,
setNoStreamError,
}: TEstablishConnection): { teardown: () => void } => {
const onRtcTrack = ({ streams }: RTCTrackEvent) => {
// We can count on there being only a single stream per event for now.
Expand Down Expand Up @@ -81,8 +84,18 @@ const establishConnection = ({
});
});
}
} else if (selectedStream && selectedStream.getAudioTracks().length === 0) {
setNoStreamError(true);
dispatch({
type: "ERROR",
payload: new Error("Stream-error: No MediaStreamTracks avaliable"),
});
} else {
// TODO handle error case of 0 available streams
setNoStreamError(true);
dispatch({
type: "ERROR",
payload: new Error("Stream-error: No MediaStream avaliable"),
});
}
};

Expand Down Expand Up @@ -219,7 +232,9 @@ export const useRtcConnection = ({
const [connectionState, setConnectionState] =
useState<RTCPeerConnectionState | null>(null);
const [audioElements, setAudioElements] = useState<HTMLAudioElement[]>([]);
const [noStreamError, setNoStreamError] = useState(false);
const audioElementsRef = useRef<HTMLAudioElement[]>(audioElements);
const navigate = useNavigate();

// Use a ref to make sure we only clean up
// audio elements once, and not every time
Expand All @@ -244,6 +259,12 @@ export const useRtcConnection = ({
[cleanUpAudio]
);

useEffect(() => {
if (noStreamError) {
navigate("/");
}
}, [navigate, noStreamError]);

useEffect(() => {
if (
!sdpOffer ||
Expand Down Expand Up @@ -286,6 +307,7 @@ export const useRtcConnection = ({
sessionId,
dispatch,
setAudioElements,
setNoStreamError,
});

return () => {
Expand All @@ -310,6 +332,7 @@ export const useRtcConnection = ({
joinProductionOptions,
rtcPeerConnection,
dispatch,
noStreamError,
]);

// Debug hook for logging RTC events TODO remove
Expand Down

0 comments on commit b28a3b1

Please sign in to comment.