Skip to content

Commit

Permalink
rtmp-services: Do not check for valid URL if using "auto"
Browse files Browse the repository at this point in the history
"Auto" is sort of a special use-case for certain services
(Twitch/Mixer), and the code recently added in be8ddc0 would
unintentionally override it.  This would for example cause the "Auto"
setting on Twitch to set the user to use an Asia server unintentionally
because "auto" is not in the json file.
  • Loading branch information
jp9000 authored and Zachary Lund committed Aug 16, 2018
1 parent 2bc1885 commit 0894623
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions plugins/rtmp-services/rtmp-common.c
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ static void ensure_valid_url(struct rtmp_common *service, json_t *json,

if (!service->server || !servers || !json_is_array(servers))
return;
if (astrcmpi(service->server, "auto") == 0)
return;

json_array_foreach (servers, index, server) {
const char *url = get_string_val(server, "url");
Expand Down

0 comments on commit 0894623

Please sign in to comment.