]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
dsdb:audit: bump minor version of password logs (1.1 -> 1.2)
authorDouglas Bagnall <douglas.bagnall@catalyst.net.nz>
Thu, 28 Aug 2025 05:09:43 +0000 (17:09 +1200)
committerDouglas Bagnall <dbagnall@samba.org>
Wed, 3 Sep 2025 02:13:40 +0000 (02:13 +0000)
In https://wiki.samba.org/index.php/Interpreting_JSON_Audit_Logs we say:

> a version number for the JSON format. It has two parts.
>
>    major: incremented if fields change meaning
>    minor: incremented if a field is added
>
> A change in possible values does not usually trigger a version
> change. This is obviously true for client supplied data, but also
> applies to e.g. passwordType, where the set of supported password
> formats can change over time without changing the JSON version.

The last paragraph explicitly exempts us from a version change for
adding a new possible value ("Public key change") to the "action"
field. On the other hand we have expanded the scope of the log, which
deserves some version recognition.

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Gary Lockyer <gary@catalyst.net.nz>
source4/dsdb/samdb/ldb_modules/audit_log.c

index b70cd04dad920484b6b4ab45822a9311ed0d177f..dd730c82e0aad22263950f27893f9d4cc88a28bf 100644 (file)
@@ -44,7 +44,7 @@
 #define PASSWORD_JSON_TYPE "passwordChange"
 #define PASSWORD_HR_TAG "Password Change"
 #define PASSWORD_MAJOR 1
-#define PASSWORD_MINOR 1
+#define PASSWORD_MINOR 2
 #define PASSWORD_LOG_LVL 5
 
 #define TRANSACTION_JSON_TYPE "dsdbTransaction"