<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, …
[View More]tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### 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 -->
- [x] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
This PR aims to update some warning levels used in `db_cluster` module to prevent silent failures.
When `db_cluster` tries to do some action in a `db_connector`, it should log some kind of warning (with `conid` that failed) or error instead of debug.
When iterating on db_connectors and one fails, it will log a warning with `conid` and if all of the available db are not available it will log an error indicating failure.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3834
-- Commit Summary --
* db_cluster: Update log levels
-- File Changes --
M src/modules/db_cluster/dbcl_api.c (83)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3834.patchhttps://github.com/kamailio/kamailio/pull/3834.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3834
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3834(a)github.com>
[View Less]
#### Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [x] 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
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
Added new Added new auth module function auth_algorithm that can be used to …
[View More]dynamically override algorithm parameter value.
#### Note
I was not able to test check README, because of this error:
```
/usr/src/orig/kamailio$ make modules-readme modules=modules/auth
make -C src/ modules-readme
make[1]: Entering directory '/usr/src/orig/kamailio/src'
make --no-print-directory -C doc auth.txt
xsltproc --novalid \
--nonet \
--novalid \
--stringparam output "auth.d" \
../../../../doc/docbook/dep.xsl auth.xml
xsltproc --novalid \
--xinclude \
../../../../doc/docbook/txt.xsl auth.xml | lynx -nolist -stdin -dump > auth.txt
error : Unknown IO error
warning: failed to load external entity "http://docbook.sourceforge.net/release/xsl/current/xhtml/docbook.xsl"
compilation error: file ../../../../doc/docbook/txt.xsl line 6 element import
xsl:import : unable to load http://docbook.sourceforge.net/release/xsl/current/xhtml/docbook.xsl
```
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3849
-- Commit Summary --
* Added new auth module function auth_algorithm
-- File Changes --
M src/modules/auth/auth_mod.c (37)
M src/modules/auth/doc/auth_functions.xml (16)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3849.patchhttps://github.com/kamailio/kamailio/pull/3849.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3849
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3849(a)github.com>
[View Less]
### Description
I can see this error during db_mysql compilation on CentOS 7
```
CC (gcc) [M db_mysql.so] km_dbase.o
CC (gcc) [M db_mysql.so] km_val.o
CC (gcc) [M db_mysql.so] km_row.o
CC (gcc) [M db_mysql.so] km_my_con.o
km_my_con.c: In function 'db_mysql_new_connection':
km_my_con.c:183:15: error: 'MYSQL_OPT_SSL_CA' undeclared (first use in this function)
ptr->con, MYSQL_OPT_SSL_CA, (const void *)db_mysql_opt_ssl_ca);
^
…
[View More]km_my_con.c:183:15: note: each undeclared identifier is reported only once for each function it appears in
make[2]: *** [km_my_con.o] Error 1
make[1]: *** [modules] Error 1
make[1]: Leaving directory `/root/rpmbuild/BUILD/kamailio-5.8.1/src'
make: *** [every-module] Error 2
```
This for 5.8.1 release.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3809
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3809(a)github.com>
[View Less]
#### Pre-Submission Checklist
- [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 …
[View More](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
- [ ] Tested changes locally
- [ ] Related to issue #3809
#### Description
fixed build on CentOS 7
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3847
-- Commit Summary --
* db_mysql: fixed build on CentOS 7
-- File Changes --
M src/modules/db_mysql/km_my_con.c (7)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3847.patchhttps://github.com/kamailio/kamailio/pull/3847.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3847
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3847(a)github.com>
[View Less]