Merged
Conversation
# Conflicts: # index.ts
…upted Refactored Python event callbacks to use a shared event dispatcher (matching the Node.js setupEventHandler pattern). Previously each of onParticipantEvent, onActiveSpeakerEvent, and onSharingEvent called super().onEventEx() independently, overwriting the previous handler. Now all event callbacks coexist through a single dispatcher. Added onMediaConnectionInterrupted callback for EVENT_MEDIA_CONNECTION_INTERRUPTED events in both Node.js and Python with auto-subscription.
Added webinar_uuid as a recognized field in JoinParams for Zoom Webinar events (webinar.rtms_started). Priority: meeting_uuid > webinar_uuid > session_id. The C SDK's rtms_join() accepts a generic meeting_uuid parameter that works for all UUID types.
…rt USER_JOIN/USER_LEAVE constants
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Release v1.0.3 of the Zoom RTMS SDK. This release adds webinar UUID support, a new
onMediaConnectionInterruptedcallback, and fixes several bugs across Node.js and Python bindings including a release crash on ended sessions, video params being overwritten by audio params, Express middleware compatibility in the webhook handler, webhook schema validation, and Python event callback coexistence.Related Issues
Fixes #93, Fixes #94, Fixes #95, Fixes #96
Type of Change
Affected Components
Testing Performed
task test:js,task test:py)docker compose run --rm test-js,docker compose run --rm test-py)req.body)webinar_uuidjoin parameteronMediaConnectionInterruptedcallback registrationrelease()no longer throws when session has already endedsetAudioParams()followed bysetVideoParams()preserves both configurationsChecklist
Summary of Changes
Bug Fixes:
release()no longer throws when the meeting has already ended;RTMS_SDK_NOT_EXISTis treated as success.setAudioParams,setVideoParams) no longer trigger default-filling logic that overwrites the other configuration.createWebhookHandlerreads fromreq.bodywhen pre-parsed by middleware likeexpress.json(), preventing hangs.eventfield is present and return 400 Bad Request for invalid payloads.onParticipantEvent,onActiveSpeakerEvent, andonSharingEventcan all be registered simultaneously.webinar_uuidparameter injoin()): Enables connecting to Zoom Webinar RTMS streams viawebinar.rtms_startedevents. Resolution priority:meeting_uuid>webinar_uuid>session_id.onMediaConnectionInterruptedcallback: High-level callback forEVENT_MEDIA_CONNECTION_INTERRUPTEDevents with auto-subscription in both Node.js and Python.USER_JOIN/USER_LEAVEconstants: Now exported for use in user update callbacks.