tm: add mandatory Max-Forwards header to local ACK

[juha-h]

Type Of Change

• [ x] Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds new functionality)
• Breaking change (fix or feature that would change existing functionality)

Checklist:

• [x ] PR should be backported to stable branches
• [ x] Tested changes locally
• Related to issue #XXXX (replace XXXX with an open issue number)

Description

Add mandatory Max-Forwards header to local ACK

[miconda]

Remember that the git commit message first line has to be formatted as per contributing guidelines, it has to be prefixed with the component name:

• https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md#commit-message-format

The commit messages are used for generating the changelog for releases as well as making easier to figure out what was impacted by a commit. Having the commit message in the proper format makes it easier to merge as well directly from the git PR portal.

[juha-h]

Same checks bug as earlier: according to Details, check-format PASSED, but there still claim that Some checks were not successful.

[linuxmaniac]

@juha-h is not a bug. The format checks each commit. I already told you how to properly do it.

Just create one commit with the proper commit message prefix squashing both previous commits in the branch.

[juha-h]

Victor Seva writes:

Just create one commit with the proper commit message prefix squashing both previous commits in the branch.

Goes beyond me. In baresip project, nothing like that needs to be done. If there is new commit to PR, all checks are automatically re-run.

[juha-h]

juha-h writes:

Goes beyond me. In baresip project, nothing like that needs to be done. If there is new commit to PR, all checks are automatically re-run.

... and red marks turn green without any extra action.

[linuxmaniac]

since it's a branch in kamailio repo I did it myself.

git rebase -i origin/master

squashed both commits and added the tm prefix

[linuxmaniac]

The idea behind it's that all commits are properly formatted.

[juha-h]

Is it OK to cherry-pick this to 5.8 branch?

[miconda]

Yes, it can be cherry-picked, it is about a mandatory header.

[juha-h]

Daniel-Constantin Mierla writes:

Yes, it can be cherry-picked, it is about a mandatory header.

Done.