Skip to content

Add log-caller-id-name option to log Caller ID Name in queue log #129

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

phoneben
Copy link

@phoneben phoneben commented Mar 1, 2025

Add log-caller-id-name option to log Caller ID Name in queue log

Fixes: #1133

UserNote:This patch adds a global configuration option, log-caller-id-name, to queues.conf to control whether the Caller ID name is logged as parameter 4 when a call enters a queue. With log-caller-id-name=yes, the queue log entry will include the URL (if specified), the caller ID number, the queue position, and—if allowed by the existing log_restricted_caller_id rules—the caller ID name. Any '|' characters in the caller ID name will be replaced with '_'.
With log-caller-id-name=no (the default), only the URL, caller ID number, and queue position are logged. Accordingly, the documented log format has been updated to:

ENTERQUEUE(url|callerid|queue_position[|calleridname])

phoneben

This comment was marked as duplicate.

@phoneben phoneben force-pushed the patch-1 branch 4 times, most recently from ef0d09c to a3303be Compare March 4, 2025 10:41
@phoneben phoneben changed the title fixing docs to be uptodate in pr#1133 Add enable_cidname_logging option to log Caller ID Name in queue log Mar 4, 2025
Copy link
Member

@jcolp jcolp left a comment

Choose a reason for hiding this comment

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

I'm not marking this as approved so it does not get merged before the code change

@phoneben phoneben changed the title Add enable_cidname_logging option to log Caller ID Name in queue log Add log_cidname option to log Caller ID Name in queue log Mar 19, 2025
Add log_cidname option to log Caller ID Name in queue log

Fixes: #1133

UserNote:This patch adds a global configuration option, log_cidname, to queues.conf to control whether the Caller ID name is logged as parameter 4 when a call enters a queue. With log_cidname=yes, the queue log entry will include the URL (if specified), the caller ID number, the queue position, and—if allowed by the existing log_restricted_caller_id rules—the caller ID name. Any '|' characters in the caller ID name will be replaced with '_'.
With log_cidname=no (the default), only the URL, caller ID number, and queue position are logged. Accordingly, the documented log format has been updated to:

ENTERQUEUE(url|callerid|queue_position[|calleridname])
@phoneben
Copy link
Author

phoneben commented Mar 19, 2025

i updated so it should be as the orig pr now .
also the old doc was missing origposition

Copy link
Member

@jcolp jcolp left a comment

Choose a reason for hiding this comment

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

Not approving until main change is merged so documentation doesn't go out of sync

Copy link
Contributor

@seanbright seanbright left a comment

Choose a reason for hiding this comment

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

This needs to be updated to show the new name of the option

@phoneben phoneben changed the title Add log_cidname option to log Caller ID Name in queue log Add log-caller-id-name option to log Caller ID Name in queue log Apr 15, 2025
@phoneben phoneben requested review from jcolp and seanbright April 15, 2025 19:29
Add log-caller-id-name option to log Caller ID Name in queue log

Fixes: #1133

UserNote:This patch adds a global configuration option, log-caller-id-name, to queues.conf to control whether the Caller ID name is logged as parameter 4 when a call enters a queue. With log-caller-id-name=yes, the queue log entry will include the URL (if specified), the caller ID number, the queue position, and—if allowed by the existing log_restricted_caller_id rules—the caller ID name. Any '|' characters in the caller ID name will be replaced with '_'.
With log-caller-id-name=no (the default), only the URL, caller ID number, and queue position are logged. Accordingly, the documented log format has been updated to:

ENTERQUEUE(url|callerid|queue_position[|calleridname])
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.

3 participants