Skip to content

[IMP] add recording feature#27

Draft
ThanhDodeurOdoo wants to merge 243 commits intomasterfrom
master-recording2-tso
Draft

[IMP] add recording feature#27
ThanhDodeurOdoo wants to merge 243 commits intomasterfrom
master-recording2-tso

Conversation

@ThanhDodeurOdoo
Copy link
Member

@ThanhDodeurOdoo ThanhDodeurOdoo commented Sep 24, 2025

I recomend reading the updated README and the associated documentation to have an overview of the code before diving in the code.

on rebase with conflicts, the files here take precendence over all previous changes (version is bumped, packages are up to date, 1.3.3 is already taken into account).

related to: odoo/odoo#234505
task-3725174

@ThanhDodeurOdoo ThanhDodeurOdoo force-pushed the master-recording2-tso branch 3 times, most recently from ac7a58e to 9d66e6e Compare October 23, 2025 13:12
ThanhDodeurOdoo added a commit to odoo-dev/odoo that referenced this pull request Nov 12, 2025
Before this commit, a failed message on the bus could
leave a promise unfulfilled.
Before this commit, if a ws authentication failed,
the ws was still registered in the connections that
need to be closed.
and harden media processing reliability

seal metadata with pre-reset audio/video/transcription flags in recorder stop flow
keep recording folders for retry on transient compile/upload failures (discard only invalid/expired)
add timeout-wrapped fetch for audio upload, routing lookup, and video upload
update/add recording tests for final-state sealing and retry-safe network failure behavior
 and use container-aware video MIME

derive video upload Content-Type from recording.videoExt (instead of hardcoded video/av1)
expose MediaOutput async init via ready and rethrow init errors after cleanup
await mediaOutput.ready in RecordingTask so PortLimitReachedError is handled reliably
make Folder.move() rethrow on rename failure so recorder fallback cleanup can run
add targeted tests for MIME selection, media init failure handling, and move-failure propagation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant