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

Spelling fixes and additional call stat reporting #798

Merged
merged 1 commit into from
Apr 24, 2023

Conversation

taclane
Copy link
Contributor

@taclane taclane commented Apr 21, 2023

A few minor fixes:

call->get_stats() should provide downstream plugins (such as MQTT) a call's number as a shorter alternative to the long id.

Calls stats should also provide the monitoringState to better identify outlier trunk recorder events such as duplicate calls or the lack of available recorders/sources. This would give users more detailed state information than state and recState currently provide.

Removed a leftover "Call_impl" reference to be consistent with other info level console text.

Corrected the spelling of the "elapsed" key used in call stats.

@robotastic
Copy link
Owner

Great upgrades!! Thanks

@robotastic robotastic merged commit 7b20e16 into robotastic:master Apr 24, 2023
@taclane taclane deleted the call_impl-stat-fix branch May 5, 2023 03:45
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.

2 participants