[Home]

Summary:ASTERISK-25422: [patch]The 'Bridge' event always reports the 'Bridgetype' as 'core' even if it's 'native'
Reporter:Birger "WIMPy" Harzenetter (wimpy)Labels:
Date Opened:2015-09-25 11:46:32Date Closed:2017-12-18 09:00:46.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/ManagerInterface
Versions:11.19.0 Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) bridgeevent_native.patch
Description:According to the documentation, the 'Bridgetype' should be reported as 'core' or 'native', however native bridges are reported as 'core'.

From a look in to the source, I don't think it's a regression, but has never worked as documented as I don't see any attempt to report the correct Bridgetype.

The patch makes the Bridge event report the correct Bridgetype while hopefully keeping the existing behavior otherwise.
It seems to work as far as I have tested.

The only change is that a failed attempt to set up a native bridge is reported as Link/native - Unlink/native - Link/core, which fits the preexisting message flow.
Comments:By: Asterisk Team (asteriskteam) 2015-09-25 11:46:33.269-0500

Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution.

A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report.

Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process].

By: Rusty Newton (rnewton) 2015-09-30 17:32:03.085-0500

Once you've followed the Code Review process [1] and submitted your code to Gerrit [2] be sure to edit this JIRA issue and add the Gerrit review URL in the appropriate field.

Thanks!

[1] https://wiki.asterisk.org/wiki/display/AST/Code+Review
[2] https://wiki.asterisk.org/wiki/display/AST/Gerrit+Usage



By: Joshua C. Colp (jcolp) 2017-12-18 09:00:46.888-0600

Bridging underwent a major rewrite which has resulted in this no longer being applicable. We have new events and we also provide different correct detail.