Thanks to visit codestin.com
Credit goes to github.com

Skip to content

chan_iax2: Add log message for rejected calls.#414

Merged
asterisk-org-access-app[bot] merged 1 commit into
asterisk:masterfrom
InterLinked1:master-issue-413
Dec 3, 2024
Merged

chan_iax2: Add log message for rejected calls.#414
asterisk-org-access-app[bot] merged 1 commit into
asterisk:masterfrom
InterLinked1:master-issue-413

Conversation

@InterLinked1

@InterLinked1 InterLinked1 commented Nov 6, 2023

Copy link
Copy Markdown
Contributor

Add a log message for a path that currently silently drops IAX2 frames without indicating that anything is wrong.

@seanbright

Copy link
Copy Markdown
Contributor

“Possible network issue?”

How is this useful?

@github-actions

github-actions Bot commented Nov 6, 2023

Copy link
Copy Markdown

REMINDER: If this PR applies to other branches, please add a comment with the appropriate "cherry-pick-to" headers as per the Create a Pull Request process.

If you don't want it cherry-picked, please add a comment with cherry-pick-to: none so we don't keep asking.

If, after adding "cherry-pick-to" comments, you change your mind, please edit the comment to DELETE the header lines and add cherry-pick-to: none.

The currently active branches are now 18, 20, 21 and master.

@InterLinked1

Copy link
Copy Markdown
Contributor Author

“Possible network issue?”

How is this useful?

It's something to go off of, and there are more comments in the code that can be used if needed.
I didn't want to get too specific like "Check /etc/hosts for non-canonical loopback addresses" because it's not guaranteed that the issue is there, that's why I tried to keep it generic.

@jcolp

jcolp commented Nov 6, 2023

Copy link
Copy Markdown
Member

This is why I suggested actually digging in more to truly understand the scenario and what is going on to provide a better and more helpful message in the correct situation.

@InterLinked1

Copy link
Copy Markdown
Contributor Author

This is why I suggested actually digging in more to truly understand the scenario and what is going on to provide a better and more helpful message in the correct situation.

If chan_iax2 isn't listening on the particular loopback address, the packets are being sent somewhere and not actually getting processed, hence the lack of a valid call number at this point. Why it is that someone is trying to dial a loopback address that isn't being listened on is really their issue to resolve, but this should clue them in on that hopefully.

@jcolp

jcolp commented Nov 6, 2023

Copy link
Copy Markdown
Member

On further looking, they were actually being received so my initial analysis was incorrect - why things went awry I don't know, but it was seemingly due to not explicitly binding. I don't think it's actually understood why it's doing what it is doing, beyond that binding explicitly causes it to work.

@seanbright seanbright left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should not be marked as fixing the associated issue. That issue should remain open.

Add a log message for a path that currently silently drops IAX2
frames without indicating that anything is wrong.
@gtjoseph

gtjoseph commented Dec 2, 2024

Copy link
Copy Markdown
Member

@InterLinked1 Cherry-pick?

@InterLinked1

Copy link
Copy Markdown
Contributor Author

cherry-pick-to: 20
cherry-pick-to: 21
cherry-pick-to: 22

@gtjoseph gtjoseph added the cherry-pick-test Trigger dry run of cherry-picks label Dec 3, 2024
@github-actions github-actions Bot added cherry-pick-testing-in-progress Cherry-Pick tests in progress cherry-pick-checks-failed Cherry-Pick checks failed and removed cherry-pick-test Trigger dry run of cherry-picks cherry-pick-testing-in-progress Cherry-Pick tests in progress labels Dec 3, 2024
@asterisk-org-access-app asterisk-org-access-app Bot merged commit d09359b into asterisk:master Dec 3, 2024
@github-actions

github-actions Bot commented Dec 3, 2024

Copy link
Copy Markdown

Successfully merged to branch master and cherry-picked to ["20","21","22"]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants