[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
RE: commit: ldap/servers/slapd/back-monitor back-monitor.h database.c init.c proto-back-monitor.h
>> -----Original Message-----
>> From: Pierangelo Masarati [mailto:ando@sys-net.it]
>
>> > Update of /repo/OpenLDAP/pkg/ldap/servers/slapd/back-monitor
>> >
>> > Modified Files:
>> > back-monitor.h 1.31 -> 1.32
>> > database.c 1.40 -> 1.41
>> > init.c 1.70 -> 1.71
>> > proto-back-monitor.h 1.16 -> 1.17
>> >
>> > Log Message:
>> > Added readOnly attribute, can be toggled on individual databases.
>>
>> Howard,
>>
>> this change is very interesting since it opens many new scenarios even
>> before back-config is available. I wonder why you don't
>> allow editing of
>> readOnly for the entire server as well; is there a specific
>> reason, or you think it's not important?
>
> A much simpler reason - if you set the Global readOnly flag, there is no
> possible way to reset it, because all subsequent LDAPModify requests
> will be rejected.
We could solve the chicken-and-egg problem by allowing this type
of modify to rootdn only; of course it has to be rootdn of the
server, not of a specific backend. I'm more and more in favor
of having a global BackendDB structure that holds global data,
making all global operations mimic those of a regular backend.
p.
--
Pierangelo Masarati
mailto:pierangelo.masarati@sys-net.it