[sr-dev] [kamailio/kamailio] statsd: remove '\r\n' from xlog messages (PR #2911)

Victor Seva notifications at github.com
Fri Oct 29 14:33:28 CEST 2021


#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, ...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] 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
- [x] 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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally

#### Description
When the module is loaded the messages have '\r\n'

> INFO: statsd [statsd.c:87]: mod_init(): Statsd init ip value 127.0.0.1 #015
> INFO: statsd [statsd.c:93]: mod_init(): Statsd init port value 8125#015
> INFO: statsd [statsd.c:101]: mod_init(): Statsd: success connection to statsd server

while at it, format the code with our clang-format definition
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * <a href="https://github.com/kamailio/kamailio/pull/2911/commits/fd9409e70401d31b16aff79bfd1eaed794aebf96">statsd: clang-format</a>
  * <a href="https://github.com/kamailio/kamailio/pull/2911/commits/8fca0fb9bf4f83ba67fcf2eec04a41e7a8c1af41">statsd: remove '\r\n' from xlog messages</a>

-- File Changes --

    M src/modules/statsd/lib_statsd.c (209)
    M src/modules/statsd/lib_statsd.h (9)
    M src/modules/statsd/statsd.c (217)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/2911.patch
https://github.com/kamailio/kamailio/pull/2911.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2911
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-dev/attachments/20211029/e7ff8028/attachment.htm>


More information about the sr-dev mailing list