> There are multiple codepaths currently, but isn't part of the
> reason for your config rewrite to rationalise that Vincent?
Huh. You're right, it's just that I, temporarily, forgot that
authentication.conf should indeed use the new implementation.
But I agree with poolie, if you have a patch to fix this (and the
related ones), no need to wait, I'll take care of the conflicts if/when
they occur and if you add tests for that, their essence should survive
the refactoring too ;)
>>>>> Martin [gz] <email address hidden> writes:
> There are multiple codepaths currently, but isn't part of the
> reason for your config rewrite to rationalise that Vincent?
Huh. You're right, it's just that I, temporarily, forgot that
authentication.conf should indeed use the new implementation.
But I agree with poolie, if you have a patch to fix this (and the
related ones), no need to wait, I'll take care of the conflicts if/when
they occur and if you add tests for that, their essence should survive
the refactoring too ;)