If I haven't overlooked something, it seems to be missing closing db connection 
on mod_init, which results in inheriting it on children after fork and 
conflicts during runtime.

Maybe it is better to move it in child_init with rank 1, we discussed a bit 
about that during K Devs meeting.

Furthermore, I recommend to add a module parameter for being able to disable 
this check, some other modules do it:

  - 
https://www.kamailio.org/docs/modules/stable/modules/auth_db.html#auth_db.p.version_table
  
It was working fine without it for many releases and it should keep a way for 
same behaviour like in the past. Disabling version table check is useful during 
upgrades, when new database table structure includes the old one, in case of 
needing to revert the upgrade for a while, running the old version is fine.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3639#issuecomment-1811931439
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/3639/c1811931...@github.com>
_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org

Reply via email to