Pre-Submission Checklist

Commit message has the format required by CONTRIBUTING guide
Commits are split per component (core, individual modules, libs, utils, ...)
Each component has a single commit (if not, squash them into one commit)
No commits to README files for modules (changes must be done to docbook files
in doc/ subfolder, the README file is autogenerated)

Type Of Change

Small 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:

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

Description

This PR fixes a bug in pseudo-variables related to "From" and "To" headers and changing the Display Name through $tu/$fu/$fn/$tn.

According to standard the URI must be enclosed in <> if a Display Name is given, but it may or may NOT if no display name is present.

So, if one tries to modify the from/to header with an initial empty display name and URI that is NOT enclosed in <>, it yields an invalid message.

This PR makes sure that if URI was not enclosed in <> and display name is modified, that will wrap URI in <>.

You can view, comment on, or merge this pull request online at:

https://github.com/kamailio/kamailio/pull/3935

Commit Summary

a43d993 pv: Ensure URI enclosed in <> when changing Display Name

File Changes

(1 file)

M src/modules/pv/pv_core.c (62)

Patch Links:

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.