Skip to content
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

debugger module: add per module facility logging + add a debug SIP message config function #175

Merged
merged 3 commits into from Jun 19, 2015

Conversation

smititelu
Copy link
Contributor

Added 2 new params to the debugger module such as the per module facility logging is now possible. Added a new config function which prints how the SIP message looks like if it would have to be sent out at that line in the config (prints the message with the applied lumps but does not apply the lumps as msg_apply changes() function).

@miconda
Copy link
Member

miconda commented Jun 1, 2015

Thanks, it will be reviewed and merged if all is ok.

Stefan Mititelu added 3 commits June 10, 2015 15:41
…g facility

Added two new debugger params which will support setting per module logging facility.
This is useful when one might want to change logging for a certain module to a different file.
mod_facility_mode (0/1) is used for enabling this. mod_facility (str) is used for
setting the facility.
Updated documentation for the new debugger facility params.
Added a config function which prints how the sip message would look like
if it were to be sent out at that point in config. Displays how the message
looks after apllying all the lumps; but it is just printing, not actual
lump application. Updated doku.
@smititelu smititelu changed the title debugger module: add per module facility logging debugger module: add per module facility logging + add a debug SIP message config function Jun 11, 2015
miconda added a commit that referenced this pull request Jun 19, 2015
debugger module: add per module facility logging + add a debug SIP message config function
@miconda miconda merged commit e0e6994 into kamailio:master Jun 19, 2015
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.

None yet

2 participants