9 Nov
2020
9 Nov
'20
11:08 p.m.
### Description
Add runtime flag -C [filename] which takes the given file name and dumps its compiled
configuration (including include and import files)
### Expected behavior
Dump complete configuration after inclusions to terminal / stdout and exit.
#### Actual observed behavior
Flag does not exist
#### Debugging Data
N/A
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
version: kamailio 5.5.0-dev3 (x86_64/linux) 76bb9b
flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST,
DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY,
USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR,
USE_DST_BLACKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED
ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535,
DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: 76bb9b
compiled on 02:45:06 Oct 24 2020 with gcc 8.3.0
--
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/issues/2546
10 Nov
10 Nov
2:42 p.m.
Just added `--cfg-print` that should do what you were looking for, to the extent of
evaluating `include_file/import_file` and `ifdef/ifndef` conditions.
It does not replace the tokens defined with `define`, `subst` or the other token defining
variants. If someone needs to go further into this, then a pull request is more than
welcome.
Basic tests were done with default config file and an imported file. If there are issues,
open a bug report.
As a side note, to be aware of, the empty lines are kept, so the output may have sequences
of several empty lines, if the `#!ifdef ... #!endif` blocks had empty lines before and
after.
--
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/issues/2546#issuecomment-724649393
2:42 p.m.
Closed #2546.
--
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/issues/2546#event-3978479083
2:47 p.m.
Very important - for now the `--cfg-print` works if the main configuration file has the
first line:
```
#!KAMAILIO
```
--
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/issues/2546#issuecomment-724651617
3:16 p.m.
Daniel-Constantin Mierla writes:
Very important - for now the `--cfg-print` works if
the main
configuration file has the first line:
#!KAMAILIO
Can there be other stuff on the line after #!KAMAILIO?
For example emacs can be told about the major mode used in the file by
checking if first non-blank line contains -*-<something>-*-.
So I would like the definition go like this:
`--cfg-print` works if the main configuration file starts with
#!KAMAILIO
-- Juha
4:37 p.m.
On 10.11.20 13:16, Juha Heinanen wrote:
Daniel-Constantin Mierla writes:
The
parsing state is switched to printing config on matching #!KAMAILIO
(and I just added the other variants #!SER, #!ALL, ...), which is
expected to be in the first line. What is before, it is not printed (but
actually parsed as usual).
I didn't want to mess very much the INITIAL state parsing, because any
token defined in cfg.lex can theoretically be in the first line, so will
make it too complex by adding conditions to each of them to check if
--cfg-print is on or not.
The other variant is to try to start in another state, then on first
char switch to INITIAL or CFGPRINTMODE, but I didn't have the time to
dig into it, I went on easy way trying not to break anything.
So for now, if someone doesn't want the #!KAMAILIO (or the other
variants) in the first line, can add it temporarily only when it uses
the --cfg-print, then remove it. Of course, if someone wants to look
more into it, is more than welcome to make pull requests.
Cheers,
Daniel
Very important - for now the `--cfg-print` works
if the main
configuration file has the first line:
#!KAMAILIO
Can there be other stuff on the line after #!KAMAILIO?
For example emacs can be told about the major mode used in the file by
checking if first non-blank line contains -*-<something>-*-.
So I would like the definition go like this:
`--cfg-print` works if the main configuration file starts with
#!KAMAILIO
--
Daniel-Constantin Mierla -- www.asipto.com
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Funding: https://www.paypal.me/dcmierla
4:52 p.m.
Daniel-Constantin Mierla writes:
So for now, if someone doesn't want the #!KAMAILIO
(or the other
variants) in the first line, can add it temporarily only when it uses
the --cfg-print, then remove it. Of course, if someone wants to look
more into it, is more than welcome to make pull requests.
It is OK to have that in the first line, but my question was, can there
be something else on the first line after #!KAMAILIO? So is the test
first_line == "#!KAMAILIO"
or
starts_with(first_line, "#!KAMAILIO")
?
-- juha
5:16 p.m.
I gave --cfg-print a try. I have this in config:
loadmodule "tls"
...
# -- tls params
#!ifdef MOD_tls
modparam("tls", "config", "/etc/sip-proxy/tls.cfg")
modparam("tls", "cipher_list", "HIGH:!ADH:!AECDH")
#!endif
# -- tm params
and --cfg-print gave this kind of output:
loadmodule "tls"
...
# -- tls params
# -- tm params
Since tls module is loaded I would have expected tls modparams to show
up in the output.
-- Juha
5:28 p.m.
On 10.11.20 15:16, Juha Heinanen wrote:
I gave --cfg-print a try. I have this in config:
loadmodule "tls"
...
# -- tls params
#!ifdef MOD_tls
modparam("tls", "config", "/etc/sip-proxy/tls.cfg")
modparam("tls", "cipher_list", "HIGH:!ADH:!AECDH")
#!endif
# -- tm params
and --cfg-print gave this kind of output:
loadmodule "tls"
...
# -- tls params
# -- tm params
Since tls module is loaded I would have expected tls modparams to show
up in the output.
It is not aware of the defines pushed from the C code, only those from
command line (e.g., -A) or config file. Maybe these ones related to
loadmodule can be caught in the lex parser, but the others pushed from
the c code after effectively loading the module are not easy to do
unless the flex parsing approach is changed. I am not planning to go
myself that way, but I am open to contributions.
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.com
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Funding: https://www.paypal.me/dcmierla
5:20 p.m.
On 10.11.20 14:52, Juha Heinanen wrote:
Daniel-Constantin Mierla writes:
Flex does token based parsing -- I haven't tried it, but it should work
even if it is something before or after it. More constraints can be
added, e.g., to be at the beginning of the line, or the only content in
the line, but currently the cfg.lex doesn't have any.
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.com
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Funding: https://www.paypal.me/dcmierla
So for now, if someone doesn't want the
#!KAMAILIO (or the other
variants) in the first line, can add it temporarily only when it uses
the --cfg-print, then remove it. Of course, if someone wants to look
more into it, is more than welcome to make pull requests.
It is OK to have that in
the first line, but my question was, can there
be something else on the first line after #!KAMAILIO? So is the test
first_line == "#!KAMAILIO"
or
starts_with(first_line, "#!KAMAILIO")
?
6:16 p.m.
Thank you!
On Tue, Nov 10, 2020 at 3:42 AM Daniel-Constantin Mierla <
notifications(a)github.com> wrote:
Closed #2546
<https://github.com/kamailio/kamailio/issues/2546>.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/kamailio/kamailio/issues/2546#event-3978479083>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AP5VMSQG7LFFXUBH3OF4E4TSPERLJANCNFSM4TPZPM5A>
.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2546#issuecomment-724768089
6:52 p.m.
I've confirmed this is working as expected, thanks again!
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2546#issuecomment-724791168
1474
days inactive
1475
days old
11 comments
4 participants
participants (4)
-
CrazyTux -
Daniel-Constantin Mierla
-
Daniel-Constantin Mierla -
Juha Heinanen