Skip to content

Commit

Permalink
obs-webrtc: Only advertise relevant codecs for a or v-only outputs
Browse files Browse the repository at this point in the history
A video-only output should not advertise any audio codecs.
  • Loading branch information
tt2468 authored and RytoEX committed May 12, 2024
1 parent 5802077 commit 09be4f9
Showing 1 changed file with 12 additions and 6 deletions.
18 changes: 12 additions & 6 deletions plugins/obs-webrtc/whip-output.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -598,6 +598,13 @@ void register_whip_output()
{
const uint32_t base_flags = OBS_OUTPUT_ENCODED | OBS_OUTPUT_SERVICE;

const char *audio_codecs = "opus";
#ifdef ENABLE_HEVC
const char *video_codecs = "h264;hevc;av1";
#else
const char *video_codecs = "h264;av1";
#endif

struct obs_output_info info = {};
info.id = "whip_output";
info.flags = OBS_OUTPUT_AV | base_flags;
Expand Down Expand Up @@ -632,21 +639,20 @@ void register_whip_output()
info.get_connect_time_ms = [](void *priv_data) -> int {
return static_cast<WHIPOutput *>(priv_data)->GetConnectTime();
};
#ifdef ENABLE_HEVC
info.encoded_video_codecs = "h264;hevc;av1";
#else
info.encoded_video_codecs = "h264;av1";
#endif
info.encoded_audio_codecs = "opus";
info.encoded_video_codecs = video_codecs;
info.encoded_audio_codecs = audio_codecs;
info.protocols = "WHIP";

obs_register_output(&info);

info.id = "whip_output_video";
info.flags = OBS_OUTPUT_VIDEO | base_flags;
info.encoded_audio_codecs = nullptr;
obs_register_output(&info);

info.id = "whip_output_audio";
info.flags = OBS_OUTPUT_AUDIO | base_flags;
info.encoded_video_codecs = nullptr;
info.encoded_audio_codecs = audio_codecs;
obs_register_output(&info);
}

0 comments on commit 09be4f9

Please sign in to comment.