Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve conference call handling #285

Merged
merged 1 commit into from
Apr 12, 2023
Merged

Improve conference call handling #285

merged 1 commit into from
Apr 12, 2023

Conversation

chenxiaolong
Copy link
Owner

@chenxiaolong chenxiaolong commented Apr 11, 2023

  • RecorderInCallService and RecorderThread have been updated to be aware of parent/child relationships of calls. Previously, child calls were treated as independent calls, so a conference call would result in separate (but identical in audio) recordings for the parent conference call and every call it was merged from.
  • RecorderInCallService and RecorderThread now trim out sections of the recording where the call is in the HOLDING state. This prevents audio from a different call from being recorded while the call is on hold (due to call waiting).
  • RecorderThread now performs a manual contact lookup if the Android telephony framework does not provide the contact display name. This is common in conference calls and is likely a bug in AOSP. As a side effect, contact lookup now works in all supported Android versions instead of just 11+.

Issue: #284

@chenxiaolong chenxiaolong self-assigned this Apr 11, 2023
@chenxiaolong
Copy link
Owner Author

Test build: BCR-1.39.r2.gebf87ed-debugOpt.zip

* RecorderInCallService and RecorderThread have been updated to be aware
  of parent/child relationships of calls. Previously, child calls were
  treated as independent calls, so a conference call would result in
  separate (but identical in audio) recordings for the parent conference
  call and every call it was merged from.
* RecorderInCallService and RecorderThread now trim out sections of the
  recording where the call is in the HOLDING state. This prevents audio
  from a different call from being recorded while the call is on hold
  (due to call waiting).
* RecorderThread now performs a manual contact lookup if the Android
  telephony framework does not provide the contact display name. This is
  common in conference calls and is likely a bug in AOSP. As a side
  effect, contact lookup now works in all supported Android versions
  instead of just 11+.

Fixes: #284

Signed-off-by: Andrew Gunnerson <[email protected]>
@chenxiaolong
Copy link
Owner Author

New test build: BCR-1.39.r2.g25a58fe-debugOpt.zip

@chenxiaolong chenxiaolong merged commit 45ec074 into master Apr 12, 2023
@chenxiaolong chenxiaolong deleted the conference branch April 12, 2023 03:01
chenxiaolong added a commit that referenced this pull request Apr 12, 2023
Signed-off-by: Andrew Gunnerson <[email protected]>
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