in principle illegal (though not causing load-time error) "obsolete_by = FALSE" definition plays a role here. 'obsolete_by' is not an boolean option.
--
The reason there's no error catched, is that obsolete_by is read with secfile_lookup_str_default(), that never returns any indication of errors - it just returns the 'default' value in these cases too.
I'd like to change it to return NULL on error (and 'default' only when there's no value given in the file being read), but that might break too many places currently relying on the return value to always be an usable string.
My comment in https://www.hostedredmine.com/issues/720559 :
--
in principle illegal (though not causing load-time error) "obsolete_by = FALSE" definition plays a role here. 'obsolete_by' is not an boolean option.
--
The reason there's no error catched, is that obsolete_by is read with secfile_lookup_str_default(), that never returns any indication of errors - it just returns the 'default' value in these cases too.
I'd like to change it to return NULL on error (and 'default' only when there's no value given in the file being read), but that might break too many places currently relying on the return value to always be an usable string.