]> git.ipfire.org Git - thirdparty/asterisk.git/commit
sorcery: Prevent duplicate objects and ensure missing objects are created on update
authorAlexei Gradinari <alex2grad@gmail.com>
Mon, 7 Jul 2025 20:52:13 +0000 (16:52 -0400)
committerGeorge Joseph <gjoseph@sangoma.com>
Wed, 17 Sep 2025 17:40:38 +0000 (11:40 -0600)
commit44034cf8419857dd280d98cac9e5bc7a8374d44b
tree07822d3018e1a85bf6fd67f9ba3bd04b869799ef
parent60b8f1d4edee0a4dc577090ef50ae49f43be9fba
sorcery: Prevent duplicate objects and ensure missing objects are created on update

This patch resolves two issues in Sorcery objectset handling with multiple
backends:

1. Prevent duplicate objects:
   When an object exists in more than one backend (e.g., a contact in both
   'astdb' and 'realtime'), the objectset previously returned multiple instances
   of the same logical object. This caused logic failures in components like the
   PJSIP registrar, where duplicate contact entries led to overcounting and
   incorrect deletions, when max_contacts=1 and remove_existing=yes.

   This patch ensures only one instance of an object with a given key is added
   to the objectset, avoiding these duplicate-related side effects.

2. Ensure missing objects are created:
   When using multiple writable backends, a temporary backend failure can lead
   to objects missing permanently from that backend.
   Currently, .update() silently fails if the object is not present,
   and no .create() is attempted.
   This results in inconsistent state across backends (e.g. astdb vs. realtime).

   This patch introduces a new global option in sorcery.conf:
     [general]
     update_or_create_on_update_miss = yes|no

   Default: no (preserves existing behavior).

   When enabled: if .update() fails with no data found, .create() is attempted
   in that backend. This ensures that objects missing due to temporary backend
   outages are re-synchronized once the backend is available again.

   Added a new CLI command:
     sorcery show settings
   Displays global Sorcery settings, including the current value of
   update_or_create_on_update_miss.

   Updated tests to validate both flag enabled/disabled behavior.

Fixes: #1289
UserNote: Users relying on Sorcery multiple writable backends configurations
(e.g., astdb + realtime) may now enable update_or_create_on_update_miss = yes
in sorcery.conf to ensure missing objects are recreated after temporary backend
failures. Default behavior remains unchanged unless explicitly enabled.

(cherry picked from commit 658b775fd65845b29810ca8c3ac9f9e3046b79d4)
configs/samples/sorcery.conf.sample
include/asterisk/sorcery.h
main/sorcery.c
res/res_sorcery_astdb.c
res/res_sorcery_memory.c
res/res_sorcery_realtime.c
tests/test_sorcery.c
tests/test_sorcery_astdb.c