[BUG] sql_cacher: improper parsing of "cache_table"

[bogdan-iancu]

The following cache definition is not accepted as valid

modparam("sql_cacher", "cache_table", "id=my_cache
db_url=mysql://opensips:opensipsrw@localhost/opensips
cachedb_url=local:///my_cache
table=my_table
key=my_key
on_demand=0
")

With the error

Nov 11 11:30:39 [3818458] ERROR:sql_cacher:parse_cache_entry: unknown parameter: 
Nov 11 11:30:39 [3818458] WARNING:sql_cacher:parse_cache_entry: invalid cache entry specification for id: my_cache
Nov 11 11:30:39 [3818458] Traceback (last included file at the bottom):
Nov 11 11:30:39 [3818458]  0. /etc/opensips/opensips.cfg
Nov 11 11:30:39 [3818458] CRITICAL:core:yyerror: parse error in /etc/opensips/opensips.cfg:255:2-3: Parameter <cache_table> not found in module <sql_cacher> - can't set

It looks like the last empty line is considered an empty name parameter :-/ . The parser should be more tolerant with such formating deviations (extra lines, extra spaces).

Valid for all versions

[github-actions]

Any updates here? No progress has been made in the last 15 days, marking as stale. Will close this issue if no further updates are made in the next 30 days.

[ar45]

Still active

I also would like to see this fixed.

[github-actions]

Any updates here? No progress has been made in the last 15 days, marking as stale. Will close this issue if no further updates are made in the next 30 days.

[ar45]

Yes

…

On Wed, Dec 15, 2021, 12:40 AM github-actions[bot] ***@***.***> wrote: Any updates here? No progress has been made in the last 15 days, marking as stale. Will close this issue if no further updates are made in the next 30 days. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2682 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABFR6GBLCA5ZQDPCJAPI52DURAZ6PANCNFSM5H2AJ5ZA> .

[github-actions]

Any updates here? No progress has been made in the last 15 days, marking as stale. Will close this issue if no further updates are made in the next 30 days.